-
Notifications
You must be signed in to change notification settings - Fork 29
Release #309
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
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Note Other AI code review bot(s) detectedCodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review. ✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
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.
Pull Request Overview
This PR is a release that includes two main enhancements to the OAuth 2.0 Password Grant authentication functionality and several supporting improvements:
- Added HTTP logging support for OAuth 2.0 authentication clients
- Fixed proxy bypass when
rp.oauth.use.proxy=falseto properly override system proxy settings - Performance optimization in
BearerAuthInterceptorby pre-computing the authorization header - Extended test coverage for log timestamp handling with microseconds support
- Updated Apache Commons Lang3 dependency from 3.18.0 to 3.19.0
Reviewed Changes
Copilot reviewed 7 out of 8 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| src/main/java/com/epam/reportportal/service/OAuth2PasswordGrantAuthInterceptor.java | Added HTTP logging interceptor setup and explicit proxy disabling with Proxy.NO_PROXY when oauthUseProxy=false |
| src/main/java/com/epam/reportportal/service/BearerAuthInterceptor.java | Performance optimization: pre-compute authorization header in constructor; updated copyright to 2025 and Apache license URL to https |
| src/main/java/com/epam/reportportal/listeners/ListenerParameters.java | Added initialization of oauthUseProxy to DEFAULT_OAUTH_USE_PROXY in constructor |
| src/test/java/com/epam/reportportal/service/OAuth2PasswordGrantAuthInterceptorTest.java | Added test for proxy bypass functionality; added MockWebServer and @Nonnull imports |
| src/test/java/com/epam/reportportal/service/LaunchMicrosecondsTest.java | Added test coverage for log timestamp handling with microseconds; updated test resource file reference |
| src/test/resources/files/responses/info_response_no_microseconds2.txt | New test resource file with version 5.12.0 response |
| build.gradle | Updated Apache Commons Lang3 from 3.18.0 to 3.19.0; added MockWebServer test dependency |
| CHANGELOG.md | Documented the two main improvements in this release |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
src/test/java/com/epam/reportportal/service/OAuth2PasswordGrantAuthInterceptorTest.java
Show resolved
Hide resolved
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
No description provided.