Merge pull request #2 from Azure-Samples/changes

Addressing PR comments in the original PR in MSAL Python repo

Author: jmprieur

1-Call-MsGraph-WithSecret/confidential_client_secret_sample.py

@@ -2,13 +2,13 @@
 The configuration file would look like this (sans those // comments):
 
 {
-    "authority": "https://login.microsoftonline.com/organizations",
+    "authority": "https://login.microsoftonline.com/Enter_the_Tenant_Name_Here",
     "client_id": "your_client_id",
     "scope": ["https://graph.microsoft.com/.default"],
         // For more information about scopes for an app, refer:
         // https://docs.microsoft.com/en-us/azure/active-directory/develop/v2-oauth2-client-creds-grant-flow#second-case-access-token-request-with-a-certificate"
 
-    "secret": "The secret generated by AAD during your confidential app registration"
+    "secret": "The secret generated by AAD during your confidential app registration",
         // For information about generating client secret, refer:
         // https://github.com/AzureAD/microsoft-authentication-library-for-python/wiki/Client-Credentials#registering-client-secrets-using-the-application-registration-portal
 
@@ -60,7 +60,7 @@
     graph_data = requests.get(  # Use token to call downstream service
         config["endpoint"],
         headers={'Authorization': 'Bearer ' + result['access_token']}, ).json()
-    print("Users from graph: " + str(graph_data))
+    print("Graph API call result: " + str(graph_data))
 else:
     print(result.get("error"))
     print(result.get("error_description"))

1-Call-MsGraph-WithSecret/parameters.json

@@ -1,5 +1,5 @@
 {
-  "authority": "https://login.microsoftonline.com/organizations",
+  "authority": "https://login.microsoftonline.com/Enter_the_Tenant_Name_Here",
   "client_id": "your_client_id",
   "scope": [ "https://graph.microsoft.com/.default" ],
   "secret": "The secret generated by AAD during your confidential app registration",

2-Call-MsGraph-WithCertificate/confidential_client_certificate_sample.py

@@ -2,14 +2,14 @@
 The configuration file would look like this (sans those // comments):
 
 {
-    "authority": "https://login.microsoftonline.com/organizations",
+    "authority": "https://login.microsoftonline.com/Enter_the_Tenant_Name_Here",
     "client_id": "your_client_id",
     "scope": ["https://graph.microsoft.com/.default"],
         // For more information about scopes for an app, refer:
         // https://docs.microsoft.com/en-us/azure/active-directory/develop/v2-oauth2-client-creds-grant-flow#second-case-access-token-request-with-a-certificate"
 
     "thumbprint": "790E... The thumbprint generated by AAD when you upload your public cert",
-    "private_key_file": "filename.pem"
+    "private_key_file": "filename.pem",
         // For information about generating thumbprint and private key file, refer:
         // https://github.com/AzureAD/microsoft-authentication-library-for-python/wiki/Client-Credentials#client-credentials-with-certificate
 
@@ -60,7 +60,7 @@
     graph_data = requests.get(  # Use token to call downstream service
         config["endpoint"],
         headers={'Authorization': 'Bearer ' + result['access_token']}, ).json()
-    print("Users from graph: " + str(graph_data))
+    print("Graph API call result: " + str(graph_data))
 else:
     print(result.get("error"))
     print(result.get("error_description"))

2-Call-MsGraph-WithCertificate/parameters.json

@@ -1,5 +1,5 @@
 {
-  "authority": "https://login.microsoftonline.com/organizations",
+  "authority": "https://login.microsoftonline.com/Enter_the_Tenant_Name_Here",
   "client_id": "your_client_id",
   "scope": [ "https://graph.microsoft.com/.default" ],
   "thumbprint": "790E... The thumbprint generated by AAD when you upload your public cert",