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

Review request for Telepathy.


Description
-------

Retrieve and save refresh token for X-MESSENGER-OAUTH2 authentication.
The refresh token will be used (in the future) to gain the access token when 
the old one is expired, so that the user can insert the password just the first 
time that he logs in.

The patch introduces the dependency on QJSON to parse the server reply, renames 
the token stored in KWallet from token to access_token, and stores the new 
refresh_token in kwallet. The authentication method is slightly different and 
requires another call to the server.

It's a bit late for 0.4, but I would like to ask for an exception, and to merge 
this part of the patch, so that perhaps I can add the second part (that will 
use the refresh token to get the new access token, and that I still need to 
write) in 0.4.1, as it won't introduce big changes. Sorry, but I was very busy 
recently...


Diffs
-----

  CMakeLists.txt e6b1df7d1017483c8234c739f84c2cc81b339882 
  cmake/modules/FindQJSON.cmake PRE-CREATION 
  x-messenger-oauth2-auth-operation.cpp 
e4aa800faecb17d12c2524d6660eb9e871cb285f 
  x-messenger-oauth2-prompt.h c87d6e7ea14c017cefd2db48d1e256802331a7eb 
  x-messenger-oauth2-prompt.cpp c987ffdbde334c010a98478a28db740ae7bde049 

Diff: http://git.reviewboard.kde.org/r/104987/diff/


Testing
-------

access_token is retrieved and correctly used to log in, refresh_token is saved.


Thanks,

Daniele Elmo Domenichelli

_______________________________________________
KDE-Telepathy mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/kde-telepathy

Reply via email to