-
Notifications
You must be signed in to change notification settings - Fork 60
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
Multiple SSO instances + browser sessions issue #524
Comments
Just noticed that the AWS SSO auth workflow is going through the default browser??? Does not honor the |
Hi @synfinatic this might be a niche use case, but I'd like to specify a different browser for auth. Is that something you could add here as well? Example:
|
@amphied Can you explain why you want to use a 3rd browser for SSO auth? I get wanting to use the default browser for SSO auth (allows re-using an existing SSO session), but not a 3rd one. |
@synfinatic I'd like to send the auth url to a script that relays it to a separate machine where my password safe and yubikey are located. Niche use case, I know :) |
So you have like two laptops? Or is one of them an EC2 or bastion host or something? |
Thank you for taking the time and answering! Two local machines. I think adding an Without that I see two workarounds:
|
Thanks for explaining... I'm trying to think about the security ramifications of this. Do you mind opening a new ticket for the ask? |
Thank you! Done: #550 |
If you have multiple SSO instances (
$AWS_SSO
) things aren't awesome because your web browser has a cookie which is tied to your SSO provider. So if you are using the same SSO provider for multiple SSO instances you can't actually switch between, because when you go through SSO and get your auth token, it is for the previous SSO provider instance- even if you do aaws-sso flush -t sso
So we either need to revert #491 (to keep the cookies in their own browser session) or do something else? Seems like we need each SSO instance to specify it's own container (or keep the default container).
The text was updated successfully, but these errors were encountered: