Prefer client_secret_post authentication method when available#191
Prefer client_secret_post authentication method when available#191guillaume-fr wants to merge 1 commit into
Conversation
|
What's the rationale for this PR? I'd appreciate a description or comments. |
|
That was a bit ago, I'll check later if I can find context. Sorry for not clarifying that earlier. TBH RFC6749 says: Including the client credentials in the request-body using the two It seems wiser to default to basic auth... Maybe I had encoding issues. If I can find anything I'll add a comment here. |
|
Likely to be an encoding issue I had with specific server. Existing code already mention switching to post by default to avoid interoperability issues: Line 149 in 8151413 If you think it's a good change, I can update the PR with a similar comment in code |
No description provided.