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

Feature request: module to Sign Requests #2146

Open
2 tasks done
cino opened this issue Feb 28, 2024 · 4 comments
Open
2 tasks done

Feature request: module to Sign Requests #2146

cino opened this issue Feb 28, 2024 · 4 comments
Labels
feature-request This item refers to a feature request for an existing or new utility need-customer-feedback Requires more customers feedback before making or revisiting a decision revisit-in-3-months Blocked issues/PRs that need to be revisited

Comments

@cino
Copy link

cino commented Feb 28, 2024

Use case

It would be nice to have a module in the AWS Powertools that would sign requests made to AWS Resources via the SigV4 method. Currently doing it yourself is a tedious process and needing to understand the underlying mechanism.

This is also discussed in the Powertools for Python repositor

Solution/User Experience

Same as for the python version, I am proposing adding a new class that will take in the parameters to create this signed request for the end user to simplify it since it is not a very straightforward method and should be made easier.

Currently it differs based on which request library you are using, for example with Axios there is an intercepter, for got there is a got4aws you could use or in cases where I'm at currently I've implemented our own hook in a request library.

Having an supported module by powertools that does this would be preferred.

Alternative solutions

No response

Acknowledgment

Future readers

Please react with 👍 and your use case to help us understand customer demand.

@cino cino added triage This item has not been triaged by a maintainer, please wait feature-request This item refers to a feature request for an existing or new utility labels Feb 28, 2024
Copy link

boring-cyborg bot commented Feb 28, 2024

Thanks for opening your first issue here! We'll come back to you as soon as we can.
In the meantime, check out the #typescript channel on our Powertools for AWS Lambda Discord: Invite link

@dreamorosi
Copy link
Contributor

Hi @cino, thank you for taking the time to open the issue.

As discussed earlier today I think this is a valuable utility, so I would definitely want to look more into it and see if we can add it to the roadmap.

I'm gonna take some time and go though the existing issues that you've linked to understand where they stand, then I'll also discuss this internally.

I'll try to get back to you here in a reasonable amount of time, but no earlier than a couple of weeks likely since we're going to focus on releasing v2 (this week) and Parser in the coming weeks.

@dreamorosi
Copy link
Contributor

An issue I opened to the AWS SDK team to improve tree shaking was addressed: smithy-lang/smithy-typescript#1159.

With v2 out of the way, I'd like to revive this conversation and work on a RFC. I have gone through the issues & PR in the Python repo and it seems that the discussion as well as implementation have stalled due to incomplete RFC and lack of traction.

Over the next couple of weeks I'll start working on a RFC and update the discussion here.

@heitorlessa heitorlessa removed the triage This item has not been triaged by a maintainer, please wait label Jun 11, 2024
@dreamorosi
Copy link
Contributor

Hi - just wanted to follow up on this.

We have been discussing this internally and we still intend to work on this utility but at the moment we are prioritizing the new Event Handler utility due to higher demand.

We'll revisit this in 3 months to see if bandwidth allows us to move this forward.

--

For future readers: ine thing that would be extremely helpful to prioritize is to have company names, as GitHub doesn't always have that info -- feel free to email us [email protected] if you don't feel comfortable sharing in public.

@dreamorosi dreamorosi added need-customer-feedback Requires more customers feedback before making or revisiting a decision revisit-in-3-months Blocked issues/PRs that need to be revisited labels Jun 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request This item refers to a feature request for an existing or new utility need-customer-feedback Requires more customers feedback before making or revisiting a decision revisit-in-3-months Blocked issues/PRs that need to be revisited
Projects
Development

No branches or pull requests

3 participants