----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103521/#review9231 -----------------------------------------------------------
x-messenger-oauth2-prompt.cpp <http://git.reviewboard.kde.org/r/103521/#comment7642> If this is all one constant, why do you construct it at runtime like this? x-messenger-oauth2-prompt.cpp <http://git.reviewboard.kde.org/r/103521/#comment7644> Hmm, this is a bit fragile... what happens if the server starts having variables in this url in a different order? x-messenger-oauth2-prompt.cpp <http://git.reviewboard.kde.org/r/103521/#comment7643> BIG BIG bug here. .data() will return the internal data pointer of QByteArray, but as soon as this line is executed, QByteArray goes out of scope and data is not a valid pointer anymore. - George Kiagiadakis 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 > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
