Skip to content

Conversation

@schnerd
Copy link
Contributor

@schnerd schnerd commented Apr 23, 2023

There are use cases where you want to pass 0 to authorize_redirect, for example Auth0 allows you to force login again by passing max_age=0, as described here.

The current code for prepare_grant_uri filters out such parameters. This PR changes that to explicitly filter out None instead.

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

  • Bugfix
  • Feature
  • Code style update
  • Refactor
  • Other, please describe:

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

Debatable? Technically a breaking change if someone was passing a non-None falsey value to authorize_redirect() previously.


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

@lepture lepture merged commit cadc55d into authlib:master Jun 7, 2023
@lepture
Copy link
Member

lepture commented Jun 7, 2023

Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants