-
Notifications
You must be signed in to change notification settings - Fork 563
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
AWS partitions support for saml2aws console
#557
base: master
Are you sure you want to change the base?
Conversation
use region= in config file as hint for where to open the console at.
I am a little concerned about littering the codebase with special cases for GovCloud and AWS China. But, I don't have the experience with the codebase (or golang itself tbh) to attempt a more comprehensive refactor. |
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.
@duckfez Are you still interested in getting this merged? If yes, please fix the conflicts and check if the code coverage is fine. If not, we can close the PR
Okta provider already has special cases saml2aws/pkg/provider/okta/okta.go Lines 652 to 654 in 1c3449a
|
I don't mind rebasing this against current main, but I no longer have access to test in GovCloud. The underlying issue #533 has not seen a lot of support for this problem, so maybe it's not worth it? Maybe one of the other interested folks in that issue can help with testing? |
Thanks for the response! Let us start by rebasing it to main. |
@duckfez Any traction on this? |
Fixes #533.
Adds support for
saml2aws console
to useregion=
in config fileas a hint for which partition to open the console in.