omarryhan/aiogoogle

Bug in `authorization_url` scopes parsing?

Closed this issue · 3 comments

This doesn't seem right to me.

scopes = scopes or client_creds["scopes"]
scopes = " ".join(client_creds["scopes"])

Should it be?

scopes = scopes or client_creds["scopes"]
scopes = " ".join(scopes)

Yeah, you're right. Nice catch!
Crazy how this was spotted after more than 2 years.
Happy to accept a PR or change it myself if you don't have the time.

Hmmmm.... thank you for the offer but for now I think it would be best if you changed it since I don't know if I can make a PR in a timely fasion.

Sure, will push a fix now.