Merge pull request #3 from cassidylaidlaw/master

ingmferrer · web-flow · commit 9edca714a13c · 2019-05-28T16:45:05.000-05:00
Fix missing state in Client.authorization_url
diff --git a/microsoftgraph/client.py b/microsoftgraph/client.py
@@ -57,7 +57,7 @@ def authorization_url(self, redirect_uri, scope, state=None):
         }
 
         if state:
-            params['state'] = None
+            params['state'] = state
         if self.office365:
             response = self.OFFICE365_AUTHORITY_URL + self.OFFICE365_AUTH_ENDPOINT + urlencode(params)
         else:

Original file line number	Diff line number	Diff line change
`@@ -57,7 +57,7 @@ def authorization_url(self, redirect_uri, scope, state=None):`
`57`	`57`	`}`
`58`	`58`
`59`	`59`	`if state:`
`60`		`- params['state'] = None`
	`60`	`+ params['state'] = state`
`61`	`61`	`if self.office365:`
`62`	`62`	`response = self.OFFICE365_AUTHORITY_URL + self.OFFICE365_AUTH_ENDPOINT + urlencode(params)`
`63`	`63`	`else:`