=== m_conley is now known as m_conley_away | ||
duanedesign | morning alll | 11:38 |
---|---|---|
karni | yo my man, duanedesign ! | 11:39 |
* mandel walking dog | 12:03 | |
nessita | hello everyone! | 12:22 |
duanedesign | hello nessita | 12:23 |
nessita | hi duanedesign! | 12:23 |
* fagan break | 12:27 | |
karni | hi nessita ! | 12:27 |
fagan | hey nessita | 12:28 |
* nessita waves | 12:28 | |
nessita | mandel: ping | 12:39 |
nessita | mandel: can you please fix what alecu asked in https://code.launchpad.net/~mandel/ubuntu-sso-client/use_txnamedpipes/+merge/61935 ? | 12:39 |
nessita | fagan: have you seen mandel around? | 12:42 |
mandel | nessita: sure | 12:47 |
mandel | nessita: I guess ou have not backlog, said I went to walk the dog :P | 12:47 |
nessita | mandel: no, I have not, sorry :-) | 12:48 |
nessita | mandel: how's the dog? | 12:48 |
mandel | nessita: noisy, smelly and dirty, so excellent well heheh | 12:49 |
nessita | hehh | 12:49 |
jussi | Hrm, I dont suppose anyone know's if joshua hoover wil be on IRC at some stage? | 12:52 |
fagan | was it fathers day in other countries yesterday? | 12:52 |
jussi | Not here. | 12:53 |
nessita | fagan: yes | 12:53 |
nessita | jussi: yes, he will be here, but in about 2 hours | 12:53 |
jussi | nessita: excellent. :) | 12:53 |
fagan | jussi: anything we can help with | 12:53 |
mandel | nessita: crazy question, do you know what is scintilla? | 12:53 |
nessita | jussi: or maybe less, 1 hour. His nickname is joshuahoover | 12:53 |
nessita | mandel: in which context? | 12:53 |
mandel | nessita: I had to patch pywin32 for our project and I have a missing lib… and I've never heard of it | 12:54 |
nessita | http://www.scintilla.org/ScintillaRelated.html ? | 12:54 |
nessita | mandel: can I see the error? | 12:54 |
mandel | nessita: it just somplains about a missing dll during a move done in the setup.py of pywin32 | 12:57 |
nessita | mandel: can I see the error? | 12:57 |
nessita | :-D | 12:57 |
mandel | nessita: sure, let me pastebin it | 12:58 |
nessita | mandel: let me know when your branch if fixed, so I also review. Also, you could please review alecu's branch? https://code.launchpad.net/~alecu/txnamedpipes/add_qt_integration/+merge/65072 | 13:00 |
mandel | nessita: ofcourse | 13:01 |
nessita | mandel: and I would like to have a quick talk with you in mumble, if possible | 13:03 |
mandel | nessita: can we do it as soon as I finished with what I'm doing right now, I wanna avoid the context switch | 13:04 |
nessita | mandel: sure | 13:04 |
mandel | nessita: what is our policy if we have to patch pywin32, shall I add the compiled bin in the tree or add the diff of the patch | 13:04 |
nessita | mandel: may I ask what is pywin32? :-) | 13:04 |
mandel | nessita: I hope the patch will be accepted (is adding a W at the end of a method) but it wont be done in time | 13:04 |
mandel | nessita: is the python extensions that allow to access the win api | 13:05 |
mandel | like win32security etc.. | 13:05 |
nessita | right | 13:05 |
nessita | so, did you fill a bug and attach the patch? | 13:05 |
mandel | nessita: I just fnished running the tests and they pass, so yeah :) | 13:06 |
mandel | nessita: but wont be ready on time for us, that I'm sure... | 13:07 |
nessita | mandel: I know, but we need to track this upstream. Do you have the link to the bug? in the mean time, I would advice you send the pywin32 version to us by email, and then we may need to distribute that in our installer | 13:07 |
mandel | nessita: ok, doing that first then we can have the mumble | 13:08 |
nessita | sure | 13:08 |
fagan | Ok I think ill look into u1 in O and see what I can fix. I know there is something with SSO and bindwood wasnt working for me which might be a couch server side thing so ill have a look at both and see what I can find out. | 13:16 |
fagan | I looked at the bug list and nothing big jumped out at me | 13:17 |
fagan | I dont think anyone else is looking at either issue so ill fiddle about and see | 13:18 |
mandel | nessita: there is just a single module to be changed in the pywin32 exetensions, so I'll just send that one rather than the full compiled bin | 13:18 |
mandel | nessita: also, do you know if there is a way to link a bug from tracker in sourceforge to lp? or to import an hg branch to an lp one so that we have something in a +junk branch just in case? | 13:19 |
mandel | nessita: is this enough for upstream: https://sourceforge.net/tracker/index.php?func=detail&aid=3323058&group_id=78018&atid=551954# | 13:20 |
ubot4 | mandel: Error: <Bugtracker.plugin.Sourceforge instance at 0x2b46ea8> bug 3323058 not found | 13:20 |
nessita | mandel: I'm not sure about bug linking | 13:20 |
nessita | mandel: I think there is a way, but you should ask in our internal channel | 13:21 |
mandel | nessita: ok on it | 13:21 |
nessita | mandel: do not forget to attach your patch to the bug report | 13:27 |
mandel | nessita: upstream one? yes, that has been done :) | 13:27 |
nessita | great! | 13:27 |
nessita | mandel: I'm ready in mumble when you are | 13:30 |
mandel | nessita: ok, one sec, I need to push a branch with the fix before I forget | 13:30 |
nessita | mandel: sure! | 13:30 |
mandel | nessita: on my way | 13:33 |
nessita | ok, I'll make some mate before the stand up | 13:52 |
* nessita brbs | 13:52 | |
mandel | nessita: udf.accentors, are those suggested paths (that is server realted ones) or local paths??? | 13:53 |
nessita | mandel: if I recall correctly, local paths | 13:57 |
nessita | yes, local paths | 13:57 |
mandel | nessita: I'm asking facundo to be 100% sure | 13:58 |
nessita | mandel: verterok is the guy to ask, but he may be off due to our National Holiday | 13:58 |
nessita | alecu, dobey, thisfred, fagan, mandel: stand up in 1' | 13:59 |
fagan | ye | 13:59 |
alecu | hellooooooo! | 13:59 |
fagan | nessita: ralsina not around today? | 13:59 |
fagan | or is he still on hols | 13:59 |
nessita | fagan: nopes, Flag Day in Argentina | 13:59 |
fagan | ahh | 13:59 |
nessita | fagan: alecu and me are swapping | 13:59 |
fagan | nessita: that explains it | 14:00 |
fagan | :) | 14:00 |
fagan | me | 14:00 |
alecu | we love to work. | 14:00 |
nessita | everything is so quite around here... there is 0 car in the street | 14:00 |
fagan | alecu: dont we all | 14:00 |
nessita | me | 14:00 |
nessita | do we? | 14:00 |
nessita | :-P | 14:00 |
nessita | hi alecu! | 14:00 |
nessita | mandel, alecu, dobey: please say me | 14:00 |
alecu | I'll do it as soon as I'm finished writting notes | 14:01 |
fagan | nessita: well I dont think id like it any more other than if my job was drinking and chasing women | 14:01 |
=== m_conley_away is now known as m_conley | ||
nessita | fagan: seriously? I think you've proven yourself that the drinking is not your thing :-D | 14:01 |
alecu | me | 14:01 |
fagan | nessita: not enough practice | 14:01 |
nessita | mandel, dobey? | 14:02 |
fagan | nessita: if I was doing it professionally id be a lot better | 14:02 |
dobey | me | 14:02 |
nessita | fagan: go! | 14:02 |
fagan | DONE | 14:02 |
fagan | * Looked at the bug list and didnt find anything that wasnt already fixed or was something that I could figure out easy enough to assign to myself. | 14:02 |
fagan | TODO | 14:02 |
fagan | * Im on 11.10 and I have a few issues so ill go look into them and see if either I can fix them or I can point out whats going wrong and where so I can ask for help or what ever. | 14:02 |
fagan | Blocked | 14:02 |
fagan | * nope | 14:02 |
fagan | nessita: go | 14:02 |
nessita | DONE: bug #798429, bug #798687, reviews, meetings | 14:02 |
nessita | TODO: bug #798964, bug #798413, reviews, windows port pushing | 14:02 |
nessita | BLOCKED: nopes | 14:02 |
nessita | NOTES: I need to go to university tomorrow instead of Wednesday, I'll be leaving after stand up and daily windows catch up meeting | 14:02 |
nessita | NEXT: alecu | 14:02 |
ubot4 | Launchpad bug 798429 in ubuntuone-control-panel "Implement functionality for adding a new cloud folder (affects: 1) (heat: 24)" [High,Fix committed] https://launchpad.net/bugs/798429 | 14:02 |
ubot4 | Launchpad bug 798687 in ubuntuone-control-panel "Migrate credentials querying to CredentialsManagementTool (affects: 1) (heat: 6)" [High,Fix committed] https://launchpad.net/bugs/798687 | 14:03 |
ubot4 | Launchpad bug 798964 in ubuntuone-control-panel "Add new folders tweaks (affects: 1) (heat: 6)" [Medium,Triaged] https://launchpad.net/bugs/798964 | 14:03 |
alecu | DONE: got u1cp-qt running on windows | 14:03 |
alecu | TODO: push all remaining branches so they can land | 14:03 |
alecu | BLOCKED: just got up, eyes still sticky | 14:03 |
alecu | NEXT: dobey | 14:03 |
ubot4 | Launchpad bug 798413 in ubuntuone-control-panel "Provide a method in backend to list only devices info without querying local settings from syncdaemon (affects: 1) (heat: 6)" [High,Triaged] https://launchpad.net/bugs/798413 | 14:03 |
dobey | λ DONE: swap day | 14:03 |
dobey | λ TODO: fix more stuff | 14:03 |
dobey | λ BLCK: None. | 14:03 |
nessita | alecu: was your face! :-) | 14:03 |
mandel | me | 14:03 |
nessita | wash* | 14:03 |
nessita | mandel: go! | 14:03 |
mandel | DONE: worked on bug #799707 which broke my sync. pywin32 has a bug in which it uses GetFileSecurity instead of GetFilesSecurityW that means that the code won't handle utf8 chars. Got caugh is some problems with the landlady and sorted it out. | 14:03 |
mandel | TODO: loads... push missing promissed proposals. Fix us_txnamedpipes in sso client. land fix fo 799707. | 14:03 |
mandel | BLOCKED: unless some other stupid life event happens, no :) | 14:03 |
ubot4 | Launchpad bug 799707 in ubuntuone-client "The windows os_helper does no have support for illegal windows paths (affects: 1) (heat: 6)" [Medium,In progress] https://launchpad.net/bugs/799707 | 14:03 |
alecu | nessita, will do! | 14:03 |
nessita | comments anyone? | 14:04 |
thisfred | me | 14:04 |
thisfred | DONE + TODO: more refinement of solution to bug #779851 / reviews BLOCKED: no | 14:04 |
ubot4 | Launchpad bug 779851 in ubuntuone-client "Ubuntu One's Unity progress bar is uninformative when transferring a single file (affects: 2) (dups: 1) (heat: 18)" [Wishlist,In progress] https://launchpad.net/bugs/779851 | 14:04 |
nessita | thisfred: GO! | 14:04 |
thisfred | sry, was making coffee, had no idea of the time | 14:04 |
nessita | thisfred: we forgive you | 14:04 |
nessita | (I think( | 14:04 |
fagan | thisfred: you werent that late :) | 14:04 |
thisfred | whew! | 14:04 |
nessita | )) | 14:04 |
nessita | ok, seems like eom to me | 14:04 |
fagan | ok on to debugging | 14:05 |
nessita | mandel: how long till you get lunch? | 14:05 |
mandel | nessita: if you wanna mumble I can wait | 14:05 |
nessita | alecu, Chipaca, mandel: mumble in 10 min? (ie 10:15ART) | 14:05 |
thisfred | LIKED: outloud.fm over turntable.fm in the end, it's just way less hassle to use | 14:06 |
* alecu is wearing a new set of clean eyes | 14:06 | |
alecu | nessita, ack | 14:06 |
* nessita will grab some abrigo, she's freezing | 14:06 | |
thisfred | alecu: better not click any links mandel sends you in that case ;) | 14:07 |
alecu | juas! | 14:07 |
mandel | nessita: I'll heat the pasta and will be back for mumble | 14:10 |
* nessita is wearing 2 pants now, and tons of coats | 14:12 | |
mterry | Chipaca, hello! I remember you saying you were working on drumming up a designer to review deja-dup. Any progress? Ideally I'd like to see the review happen this week | 14:13 |
karni | rye: Do you have your U1 gallery script url handy? | 14:13 |
rye | karni, umm, should be on the blog | 14:14 |
karni | rye: got it! | 14:14 |
karni | thanks | 14:14 |
rye | karni, lp:~rye/+junk/ubuntuone-galleria | 14:14 |
karni | http://blog.rtg.in.ua/2010/12/ubuntu-one-gallery.html | 14:14 |
karni | tnx | 14:15 |
rye | i should definitely post something | 14:15 |
duanedesign | rye: do you know of a bug report for the issue that prevents resuming downloads of files greater then 25mb? | 14:45 |
rye | duanedesign, ummm argh, i am not sure i have filed one and most likely i haven't, eventually the file will be saved, but for large files (e.g. 50+ Mbs) the process can stall completely | 14:47 |
duanedesign | i have a bug report from a user that has that isue | 14:48 |
duanedesign | bug 796877 | 14:48 |
ubot4 | Launchpad bug 796877 in ubuntuone-client "ubuntu one not syncing some files (affects: 1) (heat: 6)" [Undecided,New] https://launchpad.net/bugs/796877 | 14:48 |
duanedesign | was just curious if their was a report i could dupe it against | 14:49 |
mandel | nessita, alecu: if you can take a look at this it would be appreciated: https://code.launchpad.net/~mandel/ubuntuone-client/illegal_windows_chars/+merge/65195 | 14:51 |
* mandel moves to fix the comment from ubuntu-sso-client/use_txnamedpipes | 14:51 | |
alecu | mandel, ack | 14:52 |
nessita | mandel: ack | 14:55 |
mandel | alecu, nessita: https://code.launchpad.net/~mandel/ubuntu-sso-client/use_txnamedpipes/+merge/61935 has been updated and the unused var is gone | 14:55 |
nessita | ack | 14:56 |
* mandel moves to fix file system notifications | 14:56 | |
alecu | mandel, after that, see if you can land https://code.launchpad.net/~alecu/txnamedpipes/add_qt_integration/+merge/65072 | 14:56 |
mandel | alecu: maybe before :) | 14:56 |
mandel | alecu: just pushed the merged version to lp:txnamedpipes | 14:58 |
alecu | mandel, thanks. | 14:59 |
nessita | mandel: there is no tarmac running for txnamedpipes? | 15:01 |
nessita | mandel: easy to fix needs fixing in https://code.launchpad.net/~mandel/ubuntu-sso-client/use_txnamedpipes/+merge/61935 | 15:01 |
mandel | nessita: on it | 15:02 |
nessita | jussi: you still waiting for joshuahoover? | 15:03 |
nessita | jussi: I just saw an email from him letting us know that he won't be around today, but he will tomorrow. Can I help you somehow? | 15:03 |
mandel | nessita: just pushed an update to use_txnamedpipes without the unused imports | 15:04 |
nessita | mandel: thanks! questions regarding the other branch: | 15:04 |
nessita | mandel: was it intented that the class OSWrapperTests(BaseTwistedTestCase): disappeared? | 15:05 |
mandel | nessita: yes, since with the changes all the test in os_helper are enough since the test the same code in a more balck box manner | 15:06 |
nessita | ok | 15:06 |
mandel | nessita: know we run the tests on windows twice, with legal paths and with illegal ones | 15:06 |
nessita | mandel: how do we do that? | 15:06 |
mandel | nessita: the special tests in test_os_helper for the platform. the have the same logic but use illegal paths, I wanted to use the same, but you cannot be doing ope(test_file) where test_file is illegal, you have to ensure that you use the legal path to create it | 15:07 |
mandel | nessita: that ^ would in the case were we are testing a method that acts on an existing file (ex: rename) | 15:07 |
nessita | I see | 15:07 |
nessita | mandel: another question: | 15:08 |
nessita | how come test_path_file_exist_yes pass if you're not creating the file? | 15:08 |
nessita | mandel: nevermind | 15:08 |
nessita | I just saw the open() in setUp :-) | 15:08 |
mandel | :) | 15:08 |
dobey | la la la | 15:17 |
nessita | mandel: some needs fixing in the u1client branch | 15:22 |
thisfred | dobey: I changed some things about my progress bar branch, because I realized we could make it more accurate even. Since it changed I would love it if you could rereview. We apologize for the inconvenience. https://code.launchpad.net/~thisfred/ubuntuone-client/better-progress-bar/+merge/64887 | 15:28 |
mandel | nessita: looking | 15:28 |
dobey | thisfred: i shall grant you this one favor. and there will be a time i shall come to collect the debt. </godfather> | 15:29 |
thisfred | If I ever reproduce, you can have my firstborn | 15:30 |
fagan | thisfred: the firstborn is always the good one you dont want to give that one up | 15:33 |
fagan | go for the second one | 15:33 |
thisfred | well, I intend for it to remain academic | 15:34 |
fagan | thisfred: hah | 15:34 |
nessita | facundobatista: ping | 15:35 |
facundobatista | nessita, pong | 15:36 |
nessita | facundobatista: any clues regarding bug #798979? it has several dups | 15:36 |
ubot4 | Launchpad bug 798979 in ubuntuone-client (Ubuntu Oneiric) (and 2 other projects) "ubuntuone-syncdaemon crashed with KeyError in add_watch(): '/home/caravena/Ubuntu One' (affects: 6) (dups: 6) (heat: 56)" [High,Triaged] https://launchpad.net/bugs/798979 | 15:36 |
alecu | mandel, more needs fixing | 15:37 |
mandel | alecu: ok, looking | 15:38 |
facundobatista | nessita, we'd need debug logs | 15:38 |
* fagan break | 15:39 | |
facundobatista | nessita, but it seems that inotify is not being able to add a watch to the "Ubuntu One" dir | 15:39 |
facundobatista | nessita, why would that happen? don't know | 15:39 |
nessita | facundobatista: can you (or any otehr chicharrero) request debug logs and do some follow up, please? | 15:42 |
nessita | other* | 15:42 |
nessita | mandel: more needs fixing on ubuntu-sso-client/use_txnamedpipes/+merge/61935 | 15:44 |
mandel | nessita: ok | 15:45 |
facundobatista | nessita, is it only happening in oneiric? | 15:46 |
dobey | oh | 15:47 |
dobey | nessita, facundobatista: there is a new pyinotify in oneiric, btw | 15:47 |
facundobatista | dobey, with "new" you mean just a new version, or a "new and shiny and we broke all apis"? | 15:48 |
dobey | facundobatista: version 0.9.1 which had some changes, so that API /may/ be broken | 15:49 |
facundobatista | dobey, nessita, is the ubuntuone-client running on *any* Oneiric? | 15:49 |
nessita | facundobatista: no idea where is happening, that's why I ask you (u1foundations) to do some follow up | 15:50 |
nessita | facundobatista: nopes on desktop end (yet) | 15:50 |
dobey | facundobatista: i don't know for sure. the NM stuff should be causing breakage anyway | 15:51 |
facundobatista | dobey, what NM stuff? | 15:51 |
nessita | dobey: but mterry branch landed... | 15:52 |
dobey | facundobatista: NM 0.9 broke API, so everywhere we use NM dbus API to check for connectivity, it thinks we're offline | 15:52 |
nessita | dobey: so no breakage there, right? | 15:52 |
dobey | nessita: in trunk, not in oneiric proper | 15:52 |
facundobatista | dobey, ok, thanks | 15:52 |
dobey | nessita: and this bug report lists the oneiric proper version | 15:53 |
dobey | which reminds me, that i need to make a release of sso | 15:53 |
nessita | right. Anyway, the trace is very pyinotify specific | 15:53 |
dobey | hrmm, actually we should fix the other txsecrets issue first too :-/ | 15:53 |
dobey | oh | 15:54 |
dobey | facundobatista: but syncdaemon will still run, just never connect to the server. so it will still create watches | 15:54 |
nessita | mandel: any branch ready for re-review? | 15:54 |
dobey | facundobatista: and with new pyinotify, i guess this failure comes up :) | 15:54 |
mandel | nessita: no, I'm stuck with an error when doing one of the changes you request for u1lient | 15:55 |
nessita | mandel: ah, ok. Let me know, I'll start a branch of my own | 15:55 |
nessita | (wanted to see if I should wait to do my context swicth) | 15:55 |
dobey | thisfred: ok, so "pydoc sum" seems to be entirely useless for me understanding what sum expects as arguments, and what it returns exactly | 16:02 |
thisfred | dobey: expects a list, returns sum of values in list | 16:03 |
thisfred | well, any collection would work, I expect | 16:03 |
dobey | thisfred: well i understand the metaphorical description of what it does. i was looking for something a bit more technical though | 16:03 |
thisfred | sum=0; for i in argument: sum += i; return sum | 16:04 |
dobey | does it return a float always, an int always, or sometimes one or the other depending on content of the sequence? | 16:05 |
thisfred | though I'm not sure that's how it's implemented (this would work on generators, I'm not sure that sum does) | 16:05 |
thisfred | dobey: the latter | 16:05 |
thisfred | dobey that's just the behavior of + though | 16:05 |
thisfred | nothing to do with sum | 16:05 |
thisfred | >>> 0.1 + 1 | 16:06 |
thisfred | 1.1 | 16:06 |
thisfred | >>> 1 + 1 | 16:06 |
thisfred | 2 | 16:06 |
thisfred | >>> | 16:06 |
dobey | well, could do return float(result) or something | 16:06 |
thisfred | why though? | 16:06 |
thisfred | that would be really unexpected | 16:07 |
dobey | dude. it's python. everything it does is unexpected afaict. | 16:07 |
dobey | anyway | 16:08 |
dobey | oh | 16:08 |
dobey | your code is just hard to read because python is flippantly insane | 16:08 |
dobey | :-/ | 16:09 |
alecu | mandel, nessita: in about 20' I'll be afk for about one hour. (returning home from the in-laws) | 16:09 |
mandel | ok | 16:09 |
nessita | alecu: ack | 16:09 |
dobey | stupid python | 16:11 |
thisfred | dobey: what's the problem? I'll gladly make my code easier to read | 16:13 |
mandel | dobey: that is what she said | 16:13 |
facundobatista | dobey, what would you expect sum to do? always return a float, so lose the unbound behaviour of ints in large sums, or always return an int, truncating your floats? | 16:14 |
dobey | thisfred: the progress = float(\n bit. i had to look extra hard to see that it was really only the left-of-divisor being converted to float | 16:16 |
facundobatista | dobey, also, "floats" and "ints" are only part of the numbers that Python handles... what if you sum complex numbers, or fractions? | 16:16 |
thisfred | dobey: I can break the line at a different point I guess | 16:16 |
dobey | facundobatista: i would expect the documentation to be more explicit about what it actually does | 16:16 |
facundobatista | dobey, well, it's quite explicit: "it returns the sum of a sequence numbers" <-- it does *exactly* that | 16:17 |
=== clarita_ is now known as clarita | ||
dobey | facundobatista: i will agree to disagree. | 16:19 |
facundobatista | dobey, ok | 16:19 |
nessita | mandel: how is it going? need help? | 16:46 |
mandel | nessita: no, nearly done, I found a stupid bug thaks to your review :) | 16:46 |
mandel | was fising it | 16:46 |
nessita | :-D | 16:46 |
nessita | great!!! | 16:46 |
dobey | lunch time, bbiab | 16:50 |
nessita | thisfred: when you have a moment, could you please update https://bugs.launchpad.net/ubuntuone-control-panel/+bug/781875? (ping chad if needed, please) | 17:29 |
ubot4 | Launchpad bug 781875 in ubuntuone-control-panel "ERROR - ReplicationSettingsChangeError: args (<ubuntuone.controlpanel.dbus_service.ControlPanelBackend at /preferences (affects: 1) (heat: 5)" [Undecided,Confirmed] | 17:29 |
nessita | mandel: ping | 17:30 |
nessita | mandel: as far as I know, this bug #797258 is already fixed? if so, could you please link the branch and update status? | 17:30 |
ubot4 | Launchpad bug 797258 in ubuntuone-control-panel "The correct reactor is not used depending on the platform (affects: 1) (heat: 6)" [Undecided,New] https://launchpad.net/bugs/797258 | 17:30 |
thisfred | nessita: will do | 17:31 |
nessita | thisfred: thanks! | 17:31 |
mandel | nessita: going | 17:31 |
nessita | mandel: also, remember to use the --fixes=lp:<bug number> when submitting branches, so when the branch is scanned is attached to the bug report, and when the branch lands the bug report is marked as Fix Committed | 17:32 |
nessita | mandel: another question, do you remember what was the issue with pylint when having this issue with pyqt? http://pastebin.ubuntu.com/629907/ | 17:34 |
mandel | nessita: agg anoying, I remember the exception, there was an issue in the order of the imports, I think it was related with QtCore and QtNetwork. It was fixed by importing them in a diff order | 17:35 |
mandel | nessita: those it ring a bell? | 17:35 |
nessita | mandel: ack, thanks! | 17:35 |
nessita | yes, thanks | 17:35 |
nessita | mandel: shall I get lunch or your branches are ready for re-review? | 17:37 |
mandel | nessita: get lunch, will be ready in a few mins | 17:37 |
nessita | ack | 17:37 |
* nessita -> lunch! | 17:38 | |
* fagan EOD | 17:39 | |
mandel | nessita: in which project is bug irc://irc.freenode.net:6667/#797258 | 17:41 |
mandel | or hal where is #irc://irc.freenode.net:6667/#797258 | 17:41 |
mandel | oh, stupid me I have to say bug #irc://irc.freenode.net:6667/#797258 lame... | 17:42 |
thisfred | what irc client are *you* using :) | 17:42 |
mandel | thisfred: why? | 17:42 |
thisfred | mandel because it prefixes everything you start with a '#' with 'irc://irc.freenode.net:6667/' | 17:42 |
mandel | really? wtf adium!! | 17:43 |
thisfred | which is why ubottu is not picking it up ;) | 17:43 |
mandel | thisfred: can you tell him to do it… I cannot believe I'm ignored by a bot | 17:43 |
thisfred | bug #797258 | 17:44 |
ubot4 | Launchpad bug 797258 in ubuntuone-control-panel "The correct reactor is not used depending on the platform (affects: 1) (heat: 6)" [High,New] https://launchpad.net/bugs/797258 | 17:44 |
mandel | thisfred: txh! | 17:44 |
thisfred | maybe every bug *should* have its own irc channel, much like it has its own email address :) | 17:44 |
alecu | back | 17:46 |
dobey | thisfred: care to follow up on https://code.launchpad.net/~thisfred/ubuntuone-client/i18n-test-strings/+merge/63982 ? | 17:49 |
mandel | alecu, nessita: feel free to take a second look at https://code.launchpad.net/~mandel/ubuntuone-client/illegal_windows_chars/+merge/65195 | 17:51 |
nessita | mandel: sure | 17:53 |
mandel | alecu: your comment about using a method for the exception checking was great, the code looks a lot nicer :) | 17:54 |
alecu | mandel, :-) re-re now. | 17:54 |
=== beuno-honeymoon is now known as beuno | ||
karni | nessita: Hi hi :) \o | 17:55 |
karni | nessita: Can you tell an example for 2 errors considering one field ('email') as noted by you here https://bugs.launchpad.net/canonical-identity-provider/+bug/624955 | 17:55 |
ubot4 | Launchpad bug 624955 in canonical-isd-qa-old (and 1 other project) ""Email already registered" error is not being sent inside a list (affects: 1)" [Undecided,New] | 17:55 |
thisfred | dobey: ah yes, sry, that was still waiting somewhere in my inbox>0 | 17:55 |
nessita | karni: I'm not sure what you're asking :-) | 17:56 |
mandel | alecu, nessita: I'm away for 30 min, need to walk the dog | 17:56 |
nessita | mandel: ack | 17:56 |
karni | nessita: Sorry. You wrote in the bug report, that we expect such a reponse: {'status': 'error', 'errors': {'email': ['error1', 'error2']}} | 17:57 |
karni | nessita: Can you give me an example of 'error1' and 'error2' ? | 17:57 |
karni | nessita: I'm wondering why is it a list, and not a simple string value. | 17:57 |
karni | nessita: Also, I was a little surprized when I saw a python-like json key '__all__' (see first comment in that bug report) | 17:58 |
nessita | karni: the error report aims to unify the error reporting for any call, and maybe not for email but for other fields you may have more than one error | 17:58 |
nessita | karni: so, the report was born when I noticed that some fields will send a list of errors, and some others will send a single string | 17:59 |
karni | nessita: I see, that makes sense. | 17:59 |
alecu | mandel, there are still some instances of the old == 123 check | 17:59 |
karni | nessita: Thank you! | 17:59 |
nessita | karni: I don't recall any specific error for email other than already registered :-) | 17:59 |
nessita | you're welcome! | 17:59 |
karni | :) | 17:59 |
alecu | mandel, lines 459 and 638 in the diff. | 18:00 |
nessita | mandel: also, you didn't apply the suggestion of using os.path.splitdrive instead of all the custom logic? | 18:01 |
nessita | mandel: needs fixings added to the branch | 18:05 |
nessita | now for real, lunchtime! | 18:10 |
karni | nessita: So, the '__all__' JSON key is a catch phrase for any other errors? | 18:21 |
nessita | karni: is the key to signal errors at form level | 18:24 |
nessita | karni: not specific to any field | 18:25 |
* nessita is back | 18:25 | |
karni | nessita: Cool :) | 18:26 |
* mandel back | 18:27 | |
mandel | alecu: are those isntances exactly the same, I was very careful not to ver generalize for a reason | 18:28 |
mandel | nessita: I must have overread the split drive comment, will sort it out | 18:28 |
nessita | mandel: thanks, also please fix what I mentioned about logger.exception | 18:29 |
alecu | mandel, I don't care if you use the same function; what I meant is that there are two places with "if e.winerror and e.winerror == 123" instead of """if getattr(e, "winerror", None) == 123""" | 18:29 |
mandel | alecu: oh, that! I understood it was not using the same function… | 18:30 |
alecu | np | 18:30 |
* alecu is having a late lunch | 18:30 | |
mandel | nessita: use_txnamedpipes should not longer have pep8 issues just pushed it (r725) | 18:39 |
* mandel hates all this multiplatform things at times… is so hard to remember to check in all platfroms | 18:40 | |
nessita | mandel: ack! | 18:40 |
mandel | nessita: if you give a +1 can you do me th favour of approving the branch too, I might forget later | 18:46 |
nessita | mandel: which branch | 18:46 |
nessita | ? | 18:46 |
nessita | sso +txn? | 18:47 |
mandel | nessita: yes, sso + txn | 18:47 |
nessita | on it right now | 18:47 |
nessita | mandel: done, also I set the proposal to Approved | 18:49 |
mandel | nessita: you rule! | 18:49 |
nessita | lol | 18:49 |
nessita | Chipaca: ping | 18:50 |
nessita | Chipaca: unping | 18:50 |
nessita | mandel: which is the fastest way to implement a throbber (spinner) in QT? | 18:51 |
nessita | mandel: this is what I'm referring to http://developer.gnome.org/gtk/2.24/GtkSpinner.html | 18:52 |
* mandel looks | 18:52 | |
nessita | mandel: a solution without depending on an external animated gif will be the ideal | 18:52 |
mandel | nessita: haha I was going to propose that one :P | 18:53 |
nessita | since for that we will need the animated spinner to be provided by the design crowd | 18:53 |
nessita | mandel: and also, the spinner will be to be customized to each platform... | 18:53 |
nessita | to look native, I mean | 18:53 |
nessita | mandel: what about building a progressbar that does not show progress but only a piece that goes from left to right until the info is loaded? | 18:54 |
nessita | (not sure if I made myself clear) | 18:54 |
mandel | nessita: yes.. well usually you have to use a QMovie and a gif | 18:55 |
nessita | mandel: can't the progressbar idea be implemented without that? | 18:55 |
nessita | in GTK is what is called "activity" mode | 18:55 |
mandel | nessita: you can use a progressbar and set it to be loading, that is easy | 18:56 |
nessita | because a progress bar has different look depending on the OS as well | 18:56 |
nessita | mandel: what's the method name? I'm looking to http://doc.qt.nokia.com/latest/qprogressbar.html | 18:56 |
mandel | nessita: ok, let me find that for you | 18:57 |
nessita | so, how can I set a progressbar to be in loading/activity mode without setting explict % progress? | 18:57 |
mandel | one min | 18:57 |
nessita | sure! | 18:57 |
nessita | mandel: ah! http://old.nabble.com/ProgressBar---Activity-Mode-td29152774.html | 18:58 |
nessita | "Just set the minimum and maximum to 0 and you'll get exactly what you | 18:58 |
nessita | want. " | 18:58 |
nessita | great! thanks! | 18:58 |
nessita | :-P | 18:58 |
mandel | nessita: ha, and why dont they put that in the docs?! | 18:58 |
mandel | bastards | 18:59 |
nessita | right! | 18:59 |
mandel | nessita, alecu: please https://code.launchpad.net/~mandel/ubuntuone-client/illegal_windows_chars/+merge/65195 | 19:18 |
nessita | mandel: in 5 minutes | 19:18 |
mandel | if I forgot something is clearly telling me that I should be done for the day :P | 19:18 |
nessita | mandel: is this correct? os.chmod(fixed_path, 0o664) | 19:24 |
nessita | seems like there is an extra o there? | 19:24 |
nessita | same for os.chmod(fixed_path, 0o444) | 19:24 |
nessita | mandel: let me know when you're here, seems like we're not understanding eachother re os.path.splitdrive | 19:25 |
nessita | :-( | 19:25 |
mandel | nessita: I did use os.path.splitdrive | 19:26 |
nessita | if os.path.isabs(path): | 19:26 |
nessita | # we need to remove the volume letter, thee reason is that we | 19:26 |
nessita | # will have issues with it and the separator | 19:26 |
nessita | partial_paths = os.path.splitdrive(path)[1].split('\\') | 19:26 |
nessita | partial_paths = partial_paths[1:] | 19:26 |
nessita | path = os.path.join(*partial_paths) | 19:26 |
nessita | mandel: did you in _is_illegal_path(path)? | 19:26 |
mandel | nessita: ha, I used it wrong, I should fix that and call it a day | 19:28 |
mandel | nessita: regarding the extra o, that are the exact same values found in the os_helper from linux | 19:29 |
nessita | mandel: is there any chance you propose the SyncDaemonTool branches today? Meaning, are they proposable without any extra work on them? | 19:29 |
mandel | nessita: if I propose them we will have merge issues with the fixes regarding the notifications, from exp I prefer to propose that one first and then sd tool | 19:30 |
mandel | nessita: I can always go to the gym swim a little and get back to finish those | 19:31 |
thisfred | nessita: mandel: the 0o is because of forward compatibility, I think: http://www.python.org/dev/peps/pep-3127/ | 19:31 |
nessita | mandel: I'm worried that we're 3 days behind with the SDT, and I can guess there will be some fixing needed to those branches, so I was hoping that alecu and me could review those while you were gone, so you get feedback as soon as possible | 19:31 |
nessita | mandel: can't you propose the branches setting the proper prerequisites? | 19:32 |
mandel | nessita: yes, I can create a pipeline so that if there are fixes to commit they will be done, but before that I'll like to at least finish the current fix I'm in, otherwhise I'll screw it up later | 19:33 |
nessita | mandel: ok | 19:33 |
nessita | mandel: you let me know when the splidrive fix is ready? | 19:33 |
mandel | nessita: running the tests right now | 19:33 |
nessita | thanks! | 19:34 |
karni | nessita: Quick question. Failure on validate_email returns {'errors': {'email_token': ["Bad email token!"]}} -- should it not go with {'status':'error', 'errors' : .... } pattern? I'm asking you, because achuni is already gone from IRC :( | 19:38 |
karni | nessita: It feels inconsistent. | 19:39 |
nessita | karni: very likely to be an inconsistency | 19:39 |
karni | nessita: Smilarily, instead of returning {'status':'ok', 'email':'foo@bar.com'} it returns on success {'email':'foo@bar.com} :( | 19:40 |
nessita | yeap | 19:40 |
karni | nessita: Ok, thanks. I'll take that to ids | 19:40 |
nessita | karni: that is why we need a 2.0 API :-) | 19:40 |
karni | *isd | 19:40 |
nessita | karni: thanks! | 19:40 |
karni | nessita: Agreed!! :) | 19:40 |
mandel | nessita: os.path.splitdrive pushed | 19:45 |
nessita | mandel: ack, looking | 19:48 |
pmatulis | would someone be so kind as to demonstrate how to file sync with u1sdtool? i do '--create-foler' and it just hangs forever | 19:48 |
nessita | pmatulis: are you running u1sdtool within an X session? | 19:48 |
pmatulis | nessita: in a terminal, so i guess yeah | 19:49 |
nessita | pmatulis: a graphical terminal, right? | 19:49 |
nessita | pmatulis: if so, what does u1sdtool -s reports? | 19:50 |
pmatulis | nessita: sure, terminator | 19:50 |
pmatulis | State: READY | 19:50 |
pmatulis | queues: WORKING | 19:50 |
pmatulis | nessita: i have since stopped the create-folder comand | 19:51 |
nessita | pmatulis: so, your syncdaemon is not connected. You should first do: u1sdtool -c | 19:51 |
nessita | pmatulis: --create-folder blocks until the command is finished, and since your syncdaemon is not connected to the cloud, the command will not finish | 19:51 |
pmatulis | nessita: i did that before but i closed the resulting window. i thought that was for adding a computer | 19:52 |
pmatulis | nessita: or an account | 19:52 |
pmatulis | nessita: i have already added my computer via the web ui | 19:52 |
nessita | pmatulis: nopes, -c is "connect". It will prompt for SSO account details if there aren't any in your keyring | 19:52 |
pmatulis | nessita: hmmm | 19:52 |
nessita | pmatulis: if you have a u1client new enough, you can set the autoconnect setting to True in the conf file | 19:53 |
nessita | pmatulis: u1sdtool -c should no longer read "READY" | 19:53 |
pmatulis | problem with SSO is that you tend to forget the original p/w | 19:54 |
nessita | mandel: ping | 19:55 |
nessita | pmatulis: the same SSO screen offers password retrieval | 19:55 |
nessita | mandel: I'm about to approve, I'm running the tests now. One thing: you removed the docstring for one of the methods, can you please restore that? | 19:59 |
nessita | mandel: also, | 19:59 |
nessita | ./ubuntuone/platform/windows/ipc.py: | 19:59 |
nessita | 1246: undefined name 'get_creds_proxy' | 19:59 |
nessita | pmatulis: so, is it working now? | 19:59 |
pmatulis | nessita: i think you nailed it | 19:59 |
nessita | pmatulis: syncdaemon ir actually transferring from-to the cloud when its states is QUEUE_MANAGER + WORKING | 20:00 |
pmatulis | nessita: my previously sync'ed folder (prior to my recent re-install) is showing up in --list-folders as is my test folder | 20:00 |
pmatulis | nessita: very very nice! i have QUEUE_MANAGER and IDLE | 20:01 |
nessita | pmatulis: you're all synced up then! | 20:01 |
pmatulis | nessita: so to have this kick in upon boot what do i do? note that i'm running lubuntu | 20:01 |
dobey | it should be doing that already, assuming lubuntu follows the autostart spec | 20:03 |
nessita | dobey: I think he means the autoconnect? | 20:04 |
nessita | pmatulis: when your session starts, is it that syncdaemon is not running or is not connected? | 20:04 |
dobey | nessita: yes, and that is on by default | 20:04 |
nessita | dobey: right, let's see the version he has (autoconnect is rather new) | 20:05 |
nessita | mandel: yes? | 20:05 |
pmatulis | nessita: i haven't tested it yet as i just go it working | 20:05 |
pmatulis | nessita: but i guess autoconnect sounds good :D | 20:06 |
nessita | pmatulis: let us know. In theory, syncdaemon should autostart as a process and should autoconnect, unless you explicitly disable that setting in the conf file | 20:06 |
pmatulis | nessita: which resides where? | 20:07 |
dobey | pmatulis: what version of ubuntuone-client do you have installed? | 20:07 |
pmatulis | 1.7+r1004-36~natty1 | 20:08 |
nessita | pmatulis: ~/.config/ubuntuone/syncdaemon.conf (may not exist if you're running default settings) | 20:08 |
nessita | pmatulis: you should get the autoconnect working then | 20:09 |
pmatulis | nessita: it exists. 'on = False' | 20:10 |
nessita | pmatulis: AHA | 20:10 |
nessita | pmatulis: but wai | 20:10 |
nessita | wait | 20:10 |
nessita | pmatulis: that is throttling, by any chance? | 20:10 |
pmatulis | nessita: it does mention it, yes | 20:11 |
pmatulis | nessita: just 4 lines in there | 20:11 |
nessita | pmatulis: can you please pastebin that file? | 20:11 |
pmatulis | nessita: yes | 20:11 |
pmatulis | http://paste.ubuntu.com/629985/ | 20:12 |
alecu | mandel, approved (minus some typos on the merge proposal) | 20:12 |
mandel | nessita: I'm looking at 1246: undefined name 'get_creds_proxy' | 20:14 |
mandel | alecu: ok, I'll fix them right away | 20:14 |
nessita | pmatulis: right, so your syncdaemon is indeed set to autoconnect | 20:14 |
nessita | pmatulis: if it does not work, let us know | 20:14 |
pmatulis | nessita: nice. so i'll see tomorrow morning. thanks a lot! | 20:14 |
nessita | :-) | 20:14 |
mandel | nessita: I'm confused, what is worng with '1246: undefined name 'get_creds_proxy'' | 20:15 |
nessita | mandel: lint is complaining :-) | 20:15 |
mandel | nessita: is that on linux? | 20:16 |
nessita | yeap | 20:16 |
* mandel looks | 20:17 | |
* mandel hates multiplatform | 20:17 | |
nessita | mandel: once those are fixed, you can approve globally the merge proposal | 20:18 |
mandel | nessita: cool | 20:19 |
mandel | nessita: which method did I remove the docstring from? | 20:20 |
nessita | mandel: none, it was my mistake while reading the diff | 20:21 |
nessita | I just rechecked | 20:21 |
mandel | nessita: ok, so I'm not THAT stupid hehhe | 20:22 |
nessita | mandel: you're not stupid at all, you're just tired | 20:22 |
mandel | nessita,alecu: will fix typos and approvethen | 20:22 |
nessita | mandel: I can see why | 20:22 |
* mandel dinner, will be back shortly | 20:33 | |
czajkowski | mandel: walking the beast? | 20:39 |
alecu | nessita, mandel: https://code.launchpad.net/~alecu/ubuntuone-client/fix-credentials-defer/+merge/65267 | 21:24 |
nessita | alecu: on it!!! | 21:24 |
nessita | alecu: is the CMT working with that fix? | 21:25 |
alecu | nessita, what's CMT? Country Music Television? | 21:25 |
nessita | heh | 21:25 |
nessita | CredentialsManagementTool | 21:26 |
dobey | alecu: exactly | 21:26 |
alecu | nessita, it seems to be working, but now I'm hitting a PB/jsonrpc issue I need to debug further. | 21:26 |
nessita | alecu: did you applied mandel's branch that swicths ussoc to txn? | 21:27 |
alecu | nessita, yes, I'm using "use_txnamedpipes" on the sso | 21:27 |
alecu | nessita, but my u1cp branch still seems to want to use pb | 21:28 |
nessita | hum | 21:29 |
nessita | that smells | 21:29 |
nessita | mandel: you eod'd already? | 21:29 |
alecu | here's the full error log: http://pastebin.ubuntu.com/630022/ | 21:36 |
alecu | hmm... perhaps I need to update the local txnamedpipes | 21:37 |
dobey | have a good evening alls | 21:51 |
=== m_conley is now known as m_conley_away | ||
nessita | alecu: running tests now | 21:59 |
nessita | alecu: keeps working perfect on linux | 22:08 |
nessita | approved | 22:08 |
alecu | nessita, great. I tried it on linux, and it seems to work fine. | 22:08 |
alecu | nessita, btw: do you have any idea on this? http://pastebin.ubuntu.com/630033/ | 22:09 |
nessita | yeah, I tried it myself. No luck on windows? | 22:09 |
nessita | looking | 22:09 |
nessita | alecu: hum, I saw a branch/bug passing by, let me look that for you | 22:09 |
nessita | alecu: in any case, you need to change the import to be: | 22:09 |
nessita | from ubuntuone.platform.credentials import CredentialsManagementTool | 22:10 |
nessita | that is the proper way of using that | 22:10 |
nessita | alecu: bug #799722 | 22:11 |
ubot4 | Launchpad bug 799722 in ubuntuone-client "IPC on windows uses the old CredentialsManagementTool and has an import error (affects: 1) (heat: 6)" [Medium,In progress] https://launchpad.net/bugs/799722 | 22:11 |
nessita | hum, mandel linked the wrong branch to that bug report | 22:11 |
nessita | ah no, he actually fixed that | 22:11 |
nessita | alecu: ^ | 22:11 |
nessita | 318-from ubuntuone.credentials import CredentialsManagementTool | 22:12 |
alecu | great. | 22:12 |
nessita | 319-from ubuntuone.platform import get_creds_proxy | 22:12 |
nessita | 320+from ubuntuone.platform.windows import CredentialsManagementTool | 22:12 |
nessita | mandel: ping | 22:12 |
nessita | I guess he eoded | 22:12 |
nessita | (which makes sense) | 22:12 |
nessita | ok, I'll take a quick break to get something to eat | 22:13 |
alecu | nessita, thanks. | 22:13 |
alecu | nessita, I'm hitting an error here: http://bazaar.launchpad.net/~ubuntuone-control-tower/ubuntuone-client/trunk/view/head:/ubuntuone/platform/windows/ipc.py#L1245 | 22:28 |
alecu | nessita, this code tries to use get_creds_proxy, but it's not imported, and furthermore, it tries to register callbacks without using CMT | 22:29 |
thisfred | gotta walk the dogs, bbiab if anyone needs me | 22:29 |
nessita | alecu: looking | 22:29 |
nessita | alecu: that code looks awfully broken | 22:32 |
nessita | alecu: can you fix? | 22:32 |
alecu | nessita, sure. Any pointers? | 22:32 |
nessita | yes, let me grab/find that | 22:33 |
nessita | alecu: hum, there is no current code using CMT, but pointers are: | 22:35 |
alecu | nessita, I think I should get at the "get_creds_proxy" and then call "register_to_credentials_found" there | 22:36 |
nessita | alecu: I think the get_creds_proxy should never be used | 22:37 |
nessita | see line (from that url) 1265 | 22:37 |
nessita | the CredentialsManagementTool is created, and the deferred returned by either management.find_credentials() or management.register({'window_id': '0'}) should be used | 22:37 |
nessita | so, no signal connection should be needed | 22:37 |
alecu | nessita, oh, right. | 22:38 |
nessita | alecu: from my POV, self._register_to_signals() should be removed | 22:38 |
alecu | nessita, both find_credentials and other cmt methods already return deferreds | 22:38 |
alecu | nessita, so: you are right, we should be removing signals. | 22:38 |
nessita | yeap. So, the caller to self._request_token should handle the deferred success (the creds are returned) | 22:38 |
nessita | or deferred failures (something bad happened) | 22:38 |
alecu | nessita, great, got it now. | 22:38 |
alecu | thanks! | 22:39 |
nessita | alecu: anytime! I think you got the idea, but see line 1231, the only missing bit would be to handle errors there | 22:39 |
nessita | alecu: hum, also, the body of on_credentials_found_cb should be next to token retrieval | 22:40 |
nessita | yes? | 22:40 |
nessita | that should be all that's needed (along with the removal of _register_to_signals) | 22:40 |
alecu | got it, cool, thanks. | 22:41 |
nessita | let me know, I'll be around since my branch still needs work :-/ | 22:41 |
=== AlanChicken is now known as AlanBell | ||
=== threeve_ is now known as threeve | ||
pmatulis | nessita: i added some stuff under the directory given by --list-folders and it did not show up on the web ui. i'm on a separate system right now so i cannot troubleshoot | 23:04 |
nessita | pmatulis: first thing to check would be if the folder is subscribed. Also, please enable debug in your logs by adding this piece to your .conf file: | 23:05 |
nessita | [logging] | 23:05 |
nessita | level = DEBUG | 23:05 |
nessita | pmatulis: and then restart syncdaemon | 23:05 |
pmatulis | nessita: alright, i'll do this tomorrow morning | 23:05 |
nessita | pmatulis: let me know, I'll be around :-) | 23:06 |
pmatulis | nessita: not sure about 'subscribed', will look | 23:06 |
nessita | pmatulis: when issuing --list-folders, check the subscribed flag. Example: | 23:06 |
nessita | id=<something> subscribed=True path=/home/nessita/Documents | 23:06 |
nessita | alecu: any luck? is the CMT integration progressing? can I help? | 23:37 |
alecu | nessita, I've been working on the tests, but right now I'm writing the report and EODing | 23:38 |
nessita | alecu: makes sense, I should so myself... | 23:39 |
nessita | but this freaking stuff will not work | 23:39 |
karni | nessita: Can we ping the sso-done-so-get-tokens before we verify the email with email token? | 23:45 |
nessita | karni: nopes, that fails | 23:46 |
karni | nessita: So we need that after we validate the mail, correct? | 23:46 |
nessita | fails on server side, I mean, since that code depends on the preferred_email | 23:46 |
nessita | karni: yessir :-) | 23:46 |
karni | nessita: Roger, thank you :) | 23:46 |
nessita | you're welcome! | 23:46 |
karni | nessita: You're very helpful (and cheerful!). I like that :) | 23:47 |
nessita | thanks! :-) | 23:47 |
nessita | you're very thankful, and I appreciate that | 23:47 |
karni | ^_^ | 23:47 |
karni | nessita: Last question for today. Can we OAuth sign the ping or it needs to be Basic Authentication? | 23:49 |
karni | Oh it can't.. that's the point, to get the tokens from SSO xD | 23:49 |
karni | Right? | 23:49 |
nessita | karni: the ping has to ping OAuth signed with NO plain text, but the HMAC encoding | 23:51 |
nessita | karni: so not only it can, but it should :-) | 23:51 |
karni | nessita: Oh, it's OAuth signed! Doh, I thought Basic Auth =) Yay \o/ | 23:51 |
nessita | :-) | 23:52 |
karni | nessita: I always used Plaintext singer with Ubuntu One Files. Do you happen to know if the Files API will accept the HMAC encoding as well? | 23:55 |
nessita | karni: yes, they do. If they don't, is a huge issue. | 23:55 |
karni | nessita: Cool! Thank you =) | 23:55 |
nessita | :-) | 23:55 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!