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

AWS SigV4 Auth support #546

Merged
merged 5 commits into from
Oct 12, 2023
Merged

AWS SigV4 Auth support #546

merged 5 commits into from
Oct 12, 2023

Conversation

bmrodgers148
Copy link
Contributor

Adding support for AWS Sigv4 auth at request or collection level. Under the hood, it uses awsv4-axios to add an interceptor to the axios instance to sign the requests. Necessary additions have been made for the configuration to be made in the .bru files. No CLI support yet.

To use, just select AWS Sig V4 in the auth mode dropdown menu and enter details.
Service and Region fields are required. If Profile Name is provided, bruno will pull the credentials from the local aws credentials or config .ini files, and will ignore Access Key ID, Secret Access Key, and Session Token, as these values will be pulled from the local AWS envionment.

If there is no profile configured in the .ini files for AWS, profile name should be left blank, and Access Key ID, Secret Access Key, and optionally Session Token should be provided.

Screenshot 2023-10-11 at 5 04 53 PM

@helloanoop helloanoop merged commit faccca3 into usebruno:main Oct 12, 2023
@helloanoop
Copy link
Contributor

Thank you so much @bmrodgers148 for this huge effort to bring AWS v4 Support to Bruno! ❤️

Merged !

@mirkogolze
Copy link
Contributor

Hi,
I have seen that the usage of the AWS Auth is only used in electron. @bmrodgers148 Could you add this to the CLI too.

@bmrodgers148
Copy link
Contributor Author

@mirkogolze I'll try to add support to the CLI this week

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.

None yet

3 participants