fix: Fix list of scopes in app integrations

lepture / authlib

The ultimate Python library in building OAuth, OpenID Connect clients and servers. JWS,JWE,JWK,JWA,JWT included.

https://authlib.org/

BSD 3-Clause "New" or "Revised" License

4.39k stars 436 forks source link

fix: Fix list of scopes in app integrations #631

Open jorgeajimenezl opened 4 months ago

jorgeajimenezl commented 4 months ago

What kind of change does this PR introduce? (check at least one)

[x] Bugfix
[ ] Feature
[ ] Code style update
[ ] Refactor
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

[x] You consent that the copyright of your pull request source code belongs to Authlib's author.

codespearhead commented 3 months ago

I agree with the proposed changes, as they provide a safeguard against raising an AttributeError when scope is an iterable instead of a string. However, I can't think of a case where that will ever happen.

If there is an existing issue that prompted this pull request, please link this PR to it.