-
Notifications
You must be signed in to change notification settings - Fork 744
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
Refactor Modules for Inbound Authentication #15435
Refactor Modules for Inbound Authentication #15435
Conversation
…ina-lang into refactor-modules-for-inbound-auth
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like travis failed during test execution.
…ina-lang into refactor-modules-for-inbound-auth
Co-Authored-By: praneesha <praneesha@wso2.com> Co-Authored-By: Bhashinee <Bhashinee@users.noreply.github.com>
Codecov Report
@@ Coverage Diff @@
## master #15435 +/- ##
=========================================
Coverage ? 43.59%
Complexity ? 5393
=========================================
Files ? 983
Lines ? 38012
Branches ? 5727
=========================================
Hits ? 16571
Misses ? 19579
Partials ? 1862 Continue to review full report at Codecov.
|
Purpose
The current
ballerina/auth
module has implementation related to Config Auth Store, JWT, LDAP etc. So, with this PR,ballerina/jwt
andballerina/ldap
modules are newly created. Please refer to the discussion @ballerina-dev [1].Also, the HTTP authn handlers were renamed in a generic manner as follows:
http:JwtAuthnHandler
http:BearerAuthHeaderAuthnHandler
http:BasicAuthnHandler
http:BasicAuthHeaderAuthnHandler
[1] https://groups.google.com/d/msg/ballerina-dev/7SYDiKeF8p8/3GNncS9dAwAJ
Fixes #15385
Check List