-
Notifications
You must be signed in to change notification settings - Fork 31
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
[refactor] drop unused whoami path and drop redundant endpoints info #358
base: main
Are you sure you want to change the base?
Conversation
/test |
Aye, aye, captain! … Triggered System Test worklow! |
Git:
Job: https://github.com/drogue-iot/drogue-cloud-testing/actions/runs/3532827059 Report: https://drogue-iot.github.io/drogue-cloud-testing/test-report/2022/11/23/test-run-3532827059.html |
1 similar comment
Git:
Job: https://github.com/drogue-iot/drogue-cloud-testing/actions/runs/3532827059 Report: https://drogue-iot.github.io/drogue-cloud-testing/test-report/2022/11/23/test-run-3532827059.html |
699dd20
to
2193e4d
Compare
/test |
Aye, aye, captain! … Triggered System Test worklow! |
Git:
Job: https://github.com/drogue-iot/drogue-cloud-testing/actions/runs/4055727686 Report: https://drogue-iot.github.io/drogue-cloud-testing/test-report/2023/01/31/test-run-4055727686.html |
1 similar comment
Git:
Job: https://github.com/drogue-iot/drogue-cloud-testing/actions/runs/4055727686 Report: https://drogue-iot.github.io/drogue-cloud-testing/test-report/2023/01/31/test-run-4055727686.html |
As far as I can see, the whoami endpoint is not used anymore in the console as it pull the username from the oauth token claim.
In the case where we need to keep it (to use with an access token? Is that a use case ? ) we could move it to
/api/console/v1alpha1/user/whoami
in order to prevent the clash with the apps managements paths.