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

[CDAP-20987] OAuthHandler should return response details in the exception other than response code when request fails #15582

Merged
merged 1 commit into from
Apr 5, 2024

Conversation

priyanshu-beep
Copy link
Contributor

@priyanshu-beep priyanshu-beep commented Apr 5, 2024

This pull request enhances error handling by including additional response details in the exception when a request fails, beyond just the response code.

Testing

Screenshot 2024-04-05 at 1 53 23 PM

@priyanshu-beep priyanshu-beep added build Triggers github actions build 6.10 labels Apr 5, 2024
@itsankit-google itsankit-google removed their assignment Apr 5, 2024
@itsankit-google itsankit-google self-requested a review April 5, 2024 06:31
Copy link
Member

@itsankit-google itsankit-google left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please cherry-pick it in release/6.10 branch.

@priyanshu-beep priyanshu-beep merged commit 29e333d into develop Apr 5, 2024
13 of 16 checks passed
@priyanshu-beep priyanshu-beep deleted the story/CDAP-20987 branch April 5, 2024 08:31
priyanshu-beep added a commit that referenced this pull request Apr 5, 2024
[CDAP-20987] OAuthHandler should return response details in the exception other than response code when request fails
priyanshu-beep added a commit that referenced this pull request Apr 5, 2024
[CDAP-20987] OAuthHandler should return response details in the exception other than response code when request fails
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.10 build Triggers github actions build
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants