authlib
authlib copied to clipboard
Added client_id parameter to AssertionClient
Per https://datatracker.ietf.org/doc/html/rfc7521#section-4.1, client_id parameter, although optional, can still be passed when using assertions as authorization grants. Adding a way to pass that id to refresh token body.
What kind of change does this PR introduce? (check at least one)
- [ ] Bugfix
- [x] 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.
- I didn't see
client_idis optional in the doc. - You are always passing
client_id=None
- I didn't see
client_idis optional in the doc.- You are always passing
client_id=None
- From linked section: "Authentication of the client is optional, as described in Section 3.2.1 of OAuth 2.0 [RFC6749], and consequently, the "client_id" is only needed when a form of client authentication that relies on the parameter is used."
One of the providers I'm working with uses the authorization grant routine with client id.
- Good catch, fixed