Merge lp:~mardy/account-plugins/microsoft-graph into lp:~gary-wzl77/account-plugins/onedrive_provider

[Alberto Mardegan]

Use the microsoft graph API to get the display name

This change allows us to retrieve the user email, with a fallback to the user display name.

The Graph API to get the user info is documented here:
https://graph.microsoft.io/en-us/docs/api-reference/v1.0/resources/users

OneDrive is also accessible via the graph API (it looks like the endpoints and parameters are the same, just the base address is different), as documented here:
https://graph.microsoft.io/en-us/docs/api-reference/v1.0/resources/onedrive

[Gary.Wang (gary-wzl77)]

Thanks mardy for this change.
However, I can't manage to login with my account anymore with this change.
I'd say access token is obtained, but error occurs when retrieving the uesr email address by using graph API.

Here is the log:
...
qml: Loading changed
qml: Loading changed
qml: Loading changed
[1104/062626:ERROR:PlatformKeyboardEvent.cpp(84)] Not implemented reached in static bool blink::PlatformKeyboardEvent::currentCapsLockState()
[1104/062628:ERROR:PlatformKeyboardEvent.cpp(84)] Not implemented reached in static bool blink::PlatformKeyboardEvent::currentCapsLockState()
[1104/062631:ERROR:PlatformKeyboardEvent.cpp(84)] Not implemented reached in static bool blink::PlatformKeyboardEvent::currentCapsLockState()
qml: Loading changed
qml: Loading changed
qml: Loading changed
qml: Loading changed
qml: Loading changed
qml: Authenticated; getting cookies
QObject::disconnect: Unexpected null parameter
qml: Request destroyed!
qml: error: 401
qml: response text: {^M
  "error": {^M
    "code": "InvalidAuthenticationToken",^M
    "message": "CompactToken parsing failed with error code: -2147184118",^M
    "innerError": {^M
      "request-id": "fd40b8ca-ca3c-4bfc-84bf-58ba8b6ef0a1",^M
      "date": "2016-11-04T06:27:10"^M
    }^M
  }^M
}

review: Needs Fixing

[Gary.Wang (gary-wzl77)]

I re-tested the account plugin and found that I didn't change the Scope settings, which resulted in login failure.
Also with this plugin, access token is only available by using Microsoft graph API. So I compared all the APIs of onedrive I used in scope and storage provider
   https://dev.onedrive.com
with graph API
   https://graph.microsoft.io/en-us/docs/api-reference/v1.0/api/item_createuploadsession#best-practices)
Basically, these two sets of APIs are quite similar. With some parameters changed, I can easily migrate onedrive API to graph API.
So there's no much efforts for this transition.
Thanks for this. Mardy

review: Approve

=== modified file 'data/providers/microsoft.provider.in.in'
--- data/providers/microsoft.provider.in.in	2016-10-26 11:47:38 +0000
+++ data/providers/microsoft.provider.in.in	2016-11-03 11:10:49 +0000
@@ -17,7 +17,7 @@
           <setting name="RedirectUri">https://login.live.com/oauth20_desktop.srf</setting>
           <setting name="ResponseType">token</setting>
           <setting name="ClientId">@MICROSOFT_CLIENT_ID@</setting>
-          <setting type="as" name="Scope">['onedrive.readwrite onedrive.appfolder']</setting>          
+          <setting type="as" name="Scope">['User.Read', 'Files.ReadWrite']</setting>          
           <setting type="b" name="DisableStateParameter">true</setting>
         </group>
       </group>

=== modified file 'qml/microsoft/Main.qml'
--- qml/microsoft/Main.qml	2016-11-02 08:34:54 +0000
+++ qml/microsoft/Main.qml	2016-11-03 11:10:49 +0000
@@ -2,25 +2,26 @@
 
 OAuthMain {
     creationComponent: OAuth {
-        function getUserName(reply) {
+        function getUserName(reply, callback) {
             var http = new XMLHttpRequest();
-            var url = "https://api.onedrive.com/v1.0/drive";
+            var url = "https://graph.microsoft.com/v1.0/me";
             http.open("GET", url, true);
             http.setRequestHeader("Authorization", "bearer " + reply.AccessToken);
             http.onreadystatechange = function() {
                 if (http.readyState === 4) {
                     if (http.status === 200) {
+                        console.log("response text: " + http.responseText);
                         var response = JSON.parse(http.responseText);
-                        account.updateDisplayName(response.owner.user.displayName +
-                                                  " ( id: " + response.owner.user.id + ")");
-                        account.synced.connect(finished);
-                        account.sync();
+                        callback(response.userPrincipalName ? response.userPrincipalName : response.displayName);
                     } else {
+                        console.log("error: " + http.status);
+                        console.log("response text: " + http.responseText);
                         cancel();
                     }
                 }
             };
             http.send(null);
+            return true;
         }
-    } 
+    }
 }