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

[Request] Add support for multi-value parameters #196

Open
brikis98 opened this issue Nov 30, 2018 · 5 comments

Comments

@brikis98
Copy link

commented Nov 30, 2018

It looks like last month, API Gateway added support for multi-value params, so you can have the same header and query string params repeated multiple times with different values.

It looks like these are part of new fields on the event object called multiValueHeaders and multiValueQueryStringParameters.

Would it be possible to use these when translating the event object to a request object rather than the original, single-valued versions?

@brettstack

This comment has been minimized.

Copy link
Contributor

commented Dec 2, 2018

I have this working in a local branch as part of a larger release.

@rnowosielski

This comment has been minimized.

Copy link

commented Feb 21, 2019

@brettstack do you maybe have an idea when this might get released?

@makstaks

This comment has been minimized.

Copy link

commented Mar 26, 2019

@brettstack is this project still actively being worked on? There's a number of PRs waiting and it's unclear when the next release is coming. I'm trying to decide if I should just wait or possibly fork this project (I want to avoid forking).

@erodrig

This comment has been minimized.

Copy link

commented Apr 16, 2019

amplify uses this by default, if the project will have no maintenance from aws, they shouldn't push it . I got surprise when I saw no support for query string parameters

@albertossilva

This comment has been minimized.

Copy link

commented Apr 25, 2019

I have opened the PR #232 with this change, now I am using my fork while the PR is not merged or the branch from @brettstack get merged and published.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
6 participants
You can’t perform that action at this time.