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

Added support for multiple header value in API Gateway #16

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mattdelaf
Copy link

Last October AWS announced support for multi-value headers.

Since version 1.8.1, aws-lambda-go package events supports multi-value headers as map[string][]string.

This is particularly useful for setting multiple cookies. This is a quick fix proposal.

@tj
Copy link
Member

tj commented Jan 8, 2019

ahah silly AWS APIs..., looks good to me though, mind adding a test? I suppose we could probably always just set the multiple headers one? Since Go gives you a slice anyway

@dimiro1
Copy link

dimiro1 commented Jan 8, 2019

@davidfaynzilberg
Copy link

How do you use this library
can someone add full route request with parsed response

@tj
Copy link
Member

tj commented Nov 25, 2019

@davidfaynzilberg check out the readme, there's an example. It's basically a drop-in replacement for net/http for Lambda

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.

4 participants