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

Add helper to proxy analytics requests #15793

Merged

Conversation

chrismeyersfsu
Copy link
Member

SUMMARY
  • Handles OIDC token creation and usage transparently
ISSUE TYPE
  • New or Enhanced Feature
COMPONENT NAME
  • API
AWX VERSION
devel
ADDITIONAL INFORMATION

Copy link

codecov bot commented Jan 29, 2025

Codecov Report

Attention: Patch coverage is 97.19626% with 3 lines in your changes missing coverage. Please review.

Project coverage is 75.29%. Comparing base (ad706d6) to head (7f844d6).
Report is 8 commits behind head on devel.

✅ All tests successful. No failed tests found.

❌ Your patch status has failed because the patch coverage (97.19%) is below the target coverage (100.00%). You can increase the patch coverage or adjust the target coverage.
❌ Your project status has failed because the head coverage (75.29%) is below the target coverage (100.00%). You can increase the head coverage or adjust the target coverage.

* Handles OIDC token creation and usage transparently
@chrismeyersfsu chrismeyersfsu force-pushed the AAP-37215-analytics_proxy branch from c1d6666 to 7f844d6 Compare January 30, 2025 18:58
Copy link

@fosterseth
Copy link
Member

when OIDCClient is used in a View:

Invalid token url

500: Invalid URL 'invalidurl': No scheme supplied. Perhaps you meant https://invalidurl?

Invalid client id or secret

500: Token generation request failed

in the traceback we see
401 Client Error: Unauthorized for url: https://sso.redhat.com/auth/realms/redhat-external/protocol/openid-connect/token

Invalid scope

500: Token generation request failed

in the traceback we see
401 Client Error: Unauthorized for url: https://sso.redhat.com/auth/realms/redhat-external/protocol/openid-connect/token

@chrismeyersfsu chrismeyersfsu merged commit b53c576 into ansible:devel Jan 31, 2025
24 of 27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants