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

feat: add aws sessionToken support #12

Merged

Conversation

faisal-hameed
Copy link
Contributor

This pull request introduces support for AWS session tokens, which are necessary when using temporary security credentials. The following changes have been made:
Code Update: Added sessionToken to the configuration in index.ts to enable authentication with temporary credentials.
Documentation Update: Updated the README to include information about the sessionToken, specifying its type and usage.
These changes ensure that our application can authenticate using temporary AWS credentials, enhancing security and flexibility.

@faisal-hameed
Copy link
Contributor Author

Hi @xseman ,
I have created this PR to support sessionToken which is required when we use aws temporary credentials.
It would be great if you can review this PR.

Thanks

package.json Outdated Show resolved Hide resolved
@xseman
Copy link
Owner

xseman commented Nov 9, 2024

Thank you for your contribution! LGTM.
Once you resolve one issue and are ready, I'll merge it and release the new version.

@faisal-hameed faisal-hameed force-pushed the feature/aws-session-token-support branch from 9fdebd4 to 1c42132 Compare November 9, 2024 09:34
@xseman
Copy link
Owner

xseman commented Nov 9, 2024

I can't get test workflow to work, maybe secrets are it's limited to local branches, I tested your branch manually tests passed

@xseman xseman merged commit 77ad176 into xseman:master Nov 9, 2024
1 of 2 checks passed
@xseman xseman mentioned this pull request Nov 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants