-
-
Notifications
You must be signed in to change notification settings - Fork 404
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
[Issue #193] Integrate OAuth2 Authentication into Core Repository #676
base: master
Are you sure you want to change the base?
[Issue #193] Integrate OAuth2 Authentication into Core Repository #676
Conversation
@prayascoriolis , can you rebase or merge "master" to this PR branch please? |
…oriolis/zerocode into ISSUE-193-OAuth2-Support
@authorjapps i've merged branch "master" to this PR branch. |
ok. Thank you. Let me check if the build gets through(earlier it was stuck due to the Ubuntu image issue from GitHub Actions) |
You can check here as well @prayascoriolis : |
Thankyou, @authorjapps, All the checks have passed. |
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.
@prayascoriolis , thanks for trying to impl this.
Here are some observations and suggestions:
- What you have implemented is a "refresh token" flow
- See here how "refresh_token" flow works(Section 6): https://datatracker.ietf.org/doc/html/rfc6749#section-6
- i.e. "Refreshing an Access Token"
- What is needed is preferrably is: "access_token" flow
- See here (Section 4.4.2 and 4.4.3): https://datatracker.ietf.org/doc/html/rfc6749#section-4.4.2
- This flow should be implemented I think
- I think both flow should be supported, but "access_token" first, then "refresh_token"
@authorjapps @nirmalchandra I've completed the end to end implementation of oauth2 access token alog with refresh_token, can you please review it ? |
core/src/main/java/org/jsmart/zerocode/core/httpclient/oauth2/OAuth2Impl.java
Outdated
Show resolved
Hide resolved
core/src/main/java/org/jsmart/zerocode/core/httpclient/oauth2/OAuth2HttpClient.java
Outdated
Show resolved
Hide resolved
…se, and dangling Javadoc comment fixed.
…oriolis/zerocode into ISSUE-193-OAuth2-Support
[Issue #193] Implement OAuth2 Authentication in Core Repository
Fixes Issue: 193
PR Branch
https://github.com/prayascoriolis/zerocode/tree/ISSUE-193-OAuth2-Support
Motivation and Context
The integration of OAuth2 authentication into the core repository is a key enhancement that will make the library more versatile and secure. By aligning this effort with the successful example from the zerocode-hello-world repository, you are paving the way for a robust authentication mechanism that can be leveraged across various use cases.
PR Description:
This pull request integrates OAuth2 authentication into the core repository. The following tasks have been undertaken:
Checklist:
New Unit tests were added
Integration tests were added
Test names are meaningful
Feature manually tested and outcome is successful
PR doesn't break any of the earlier features for end users
Branch build passed in CI
No 'package.*' in the imports
Http test added to
http-testing
module(if applicable)Not applicable. The changes did not affect Kafka automation flow