Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support for mTLS #420

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ricklambrechts
Copy link
Contributor

@ricklambrechts ricklambrechts commented May 26, 2024

I have added support for mTLS client authentication [RFC8705].

With the new added getEndpoint function the correct (mtls) endpoint will be loaded from the provider config.
Added necessary variables for holding the mtls configuration, if wanted I can move it to a MtlsConfig class for example.

TODO:

  • Use only authentication method
  • Only for token, introspection, revocation and backchannel authentication endpoints.

List of common tasks a pull request require complete

  • Changelog entry is added or the pull request don't alter library's functionality

@DeepDiver1975
Copy link
Collaborator

this is yet another rare usage case from my understanding. can we support such an implementation via a subclassing and overloading the required methods?

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