-
Notifications
You must be signed in to change notification settings - Fork 5
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
Implement ingest-api auth login from swagger #347
Comments
UPDATE Working
Needs work Issue comment that inspired the spike This snippet adds the desired callback url but stomps the rest of the hosted ui configuration including existing callback urls
These important configs for the Hosted UI get wiped out by the above AwsSdkCall |
### Issue NASA-IMPACT/veda-data-airflow#134 #347 ### What? - Update get_username to fall back on `sub` if `username` doesn't exist in token - Added some more logging statements to make debugging easier in the future ### Why? - This fix is to enable a successful workflows API run since the workflow API passes a token to backend API and it's currently erroring - Also, `sub` is a more definitive identifier because it represents a unique identifier compared to `username` ### Testing? - Relevant testing details
What
Currently administrators must manually post username and password to a token endpoint and copy paste the token from the response for ingest operations. Update this auth flow to follow the more standard redirect to auth provider for a secure username and password form entry and redirect to the swagger docs.
AC
The text was updated successfully, but these errors were encountered: