Skip to content
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

Use of SSO roles for PHP CLI #2848

Open
1 of 2 tasks
exussum12 opened this issue Nov 29, 2023 · 1 comment
Open
1 of 2 tasks

Use of SSO roles for PHP CLI #2848

exussum12 opened this issue Nov 29, 2023 · 1 comment
Labels
feature-request A feature should be added or improved. needs-review p3 This is a minor priority issue

Comments

@exussum12
Copy link

exussum12 commented Nov 29, 2023

Describe the feature

I am hoping to follow this workflow

aws configure sso or aws sso login --profile=something

Commands on the AWS CLI work, I am hoping to be able to use the same profile (something in the case above) to be able to reuse in php scripts

Use Case

Removing long lived keys for local development

Proposed Solution

The sso logs to a few files (~/.aws/sso/cache)

These should be scanned when looking for a profile being passed current behaviour is
PHP Fatal error: Uncaught Aws\Exception\CredentialsException: 'something' not found in credentials file in /tmp/aws/vendor/aws/aws-sdk-php/src/Credentials/CredentialProvider.php:826

Other Information

This should ideally work for rules assumed via other accounts. That has a seperate storage (.aws/cli/cache)

Acknowledgements

  • I may be able to implement this feature request
  • This feature might incur a breaking change

SDK version used

Latest

Environment details (Version of PHP (php -v)? OS name and version, etc.)

php 8.2

@exussum12 exussum12 added feature-request A feature should be added or improved. needs-triage This issue or PR still needs to be triaged. labels Nov 29, 2023
@exussum12
Copy link
Author

exussum12 commented Nov 29, 2023

So this does work as follows

'credentials' =>  CredentialProvider::sso('something'),

In a construct, not sure if the profile should pick it up? That is how the CLI works

Digging into this some more, if a profile is passed the ini loader is called, which does not support SSO. Any objections to moving that to a chain? First a standard profile and fall back to a SSO profile

@yenfryherrerafeliz yenfryherrerafeliz added needs-review p3 This is a minor priority issue and removed needs-triage This issue or PR still needs to be triaged. labels Feb 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request A feature should be added or improved. needs-review p3 This is a minor priority issue
Projects
None yet
Development

No branches or pull requests

2 participants