Fix scopes process of get_authorization_url

[obsd]

The function get_authorization_url is passing scopes to get_session without process, so you get an error that there aren't such scopes.
You can see here how the scopes look like before the fix and compare them to the scopes that did work:

print(requested_scopes, self.scopes)
['basic', 'message_all'] ['https://graph.microsoft.com/Mail.ReadWrite', 'https://graph.microsoft.com/Mail.Send', 'offline_access', 'https://graph.microsoft.com/User.Read']

and after the fix:

protocol = MSGraphProtocol()
protocol.get_scopes_for(requested_scopes)
['https://graph.microsoft.com/Mail.ReadWrite', 'https://graph.microsoft.com/Mail.Send', 'offline_access', 'https://graph.microsoft.com/User.Read']

[odedvisiblerisk]

@janscas can you review it?

[alejcas]

Hi!
I know, but this is on purpose. The connection is "protocol" agnostic. It doesn't know the protocol used and you can't assume is MsGraphProtocol.

This is somehow a design decision made a long time ago.
For this reason, when not using the Account object, you should use a Connection and a Protocol.
Pass the scopes to the protocol get_scopes_for method and then pass the returned scopes to the Connection (or directly to the method get_authorization_urlif you prefer).

Thanks!

[alejcas]

You can't assume here that the protocol in use is MSGraphProtocol.