-
Notifications
You must be signed in to change notification settings - Fork 54
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
Support SSO #51
Comments
I have fixed this by upgrading to the latest version of the aws sdk changes. Created new branch with changes, but don't have permission to push |
@davidsteed If you have SSO working, can you submit a pull request with the code change? |
Yes all I did was the following
go get -u github.com/aws/aws-sdk-go
go mod vendor
Then built the code. I had to change the Makefile as it does not work on
Windows bash.
I was going to submit the pull request, but can't run the automated tests
as the way go genrate works also does not work on windows bash - I get my
mac back next week.
The image does appear to work though
…On Tue, 9 Feb 2021 at 19:10, Wesley Pettit ***@***.***> wrote:
@davidsteed <https://github.com/davidsteed> If you have SSO working, can
you submit a pull request with the code change?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#51 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AECIHJZDWQPGVSW7LRTBNITS6GCDDANCNFSM4XKUNIDA>
.
|
I can confirm @davidsteed is correct, and I am also currently struggling with the I'll submit a PR (also) if I can get this working and passing tests. For now, it's perfectly functional for our immediate AWS SSO needs. |
I just got this working locally with all tests passing, PR incoming. The root of the issue occurs when running |
I merged #52 This project isn't super actively maintained so not sure when we will get around to doing a release. |
THANK YOU! Even without a formal release, this is a really big feature for us to carry only a custom binary instead of a custom fork + custom binary. I'll be passing the positive feedback along to our TAM, and finally getting our customer commentary for the AWS SSO team. |
Hello, is there any planned release including this change? As an alternative, how can I build the docker image locally? |
Any chance of a release? |
@Kaelten @arnaudmm @masteinhauser I think the latest release should include it: https://github.com/awslabs/amazon-ecs-local-container-endpoints/releases/tag/v1.4.0 Based on the fact that the release was cut after I merged the SSO PR. I checked the commit history. |
I'm still getting the below when I try to use 1.4.0
|
scratch that, it turns out I wasn't actually using 1.4.0 since I was trying to use |
A slightly different solution to this and any other credentialing woes is to provide refreshable credentials directly from the host via an IMDS server, which I've elaborated on in #171 |
This does not support "aws configure sso". If I copy the credentials into the ~/.aws/credentials file it works. Can support for sso be added
The text was updated successfully, but these errors were encountered: