Review Request: Support X-MESSENGER-OAUTH2 authentication mechanism and make dialogs modeless

[Daniele Elmo Domenichelli]

> On Dec. 24, 2011, 5:40 p.m., Dario Freddi wrote:
> > x-messenger-oauth2-auth-operation.cpp, lines 65-68
> > <http://git.reviewboard.kde.org/r/103521/diff/1/?file=44654#file44654line65>
> >
> >     I see a potential focus issue here, given you are constructing the dialog without a parent at all. This might lead to the dialog appearing behind the windows in some cases, and failure in having the user properly notified. Is it possible to give it a parent somehow, maybe passing a widget to the operation?

We don't have any parent, the service is running in the background without any widget and the window just pops up like the password ones, so we don't have any widget to pass...
Do you have any suggestion about how to do this?


- Daniele Elmo


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103521/#review9239
-----------------------------------------------------------


On Dec. 23, 2011, 3:25 p.m., Daniele Elmo Domenichelli wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103521/
> -----------------------------------------------------------
> 
> (Updated Dec. 23, 2011, 3:25 p.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Description
> -------
> 
> Refactor SaslAuthOp to support different authentication methods, support X-MESSENGER-OAUTH2 authentication mechanism and make dialogs modeless
> 
> branch here: http://quickgit.kde.org/?p=scratch%2Fddomenichelli%2Ftelepathy-auth-handler.git&a=shortlog&h=refs/heads/oauth2
> 
> At the moment there is no support from the kcm, therefore you will have to add to your ~/.mission-control/accounts/accounts.cfg something like this:
> 
> [gabble/jabber/messenger_2elive_2ecom0]
> manager=gabble
> protocol=jabber
> param-server=xmpp.messenger.live.com
> param-account=messenger.live.com
> param-ignore-ssl-errors=true
> 
> 
> This addresses bugs 283307 and 289668.
>     http://bugs.kde.org/show_bug.cgi?id=283307
>     http://bugs.kde.org/show_bug.cgi?id=289668
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 1a263b66349074e8d409adfefd7d53af7d05a07d 
>   password-prompt.h fc686ca4be3cc88cc7f968f6fca6f8a2c89019fe 
>   password-prompt.cpp e23842913c33efce29ff9cc1f474c7acf0449c30 
>   password-prompt.ui 7f5009e0f473bdb3c70a600b70a7f12b8fa6827d 
>   sasl-auth-op.h e58543b6de9b50e05f6d6c5e3ccabf6fa90bc2f3 
>   sasl-auth-op.cpp b65baacc4afa1323fd2392f621b3944d702138c1 
>   sasl-handler.cpp ad34e37e7f757dac1fda95c0f8e8dbece1aae9af 
>   x-messenger-oauth2-auth-operation.h PRE-CREATION 
>   x-messenger-oauth2-auth-operation.cpp PRE-CREATION 
>   x-messenger-oauth2-prompt.h PRE-CREATION 
>   x-messenger-oauth2-prompt.cpp PRE-CREATION 
>   x-telepathy-password-auth-operation.h PRE-CREATION 
>   x-telepathy-password-auth-operation.cpp PRE-CREATION 
>   x-telepathy-password-prompt.h PRE-CREATION 
>   x-telepathy-password-prompt.cpp PRE-CREATION 
>   x-telepathy-password-prompt.ui PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/103521/diff/diff
> 
> 
> Testing
> -------
> 
> Can login to MSN using gabble/xmpp
> Can still login using old method (either with password prompt or stored password)
> 
> 
> Thanks,
> 
> Daniele Elmo Domenichelli
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20111226/0ebead2b/attachment-0001.html>