Skip to content
This repository has been archived by the owner on Aug 27, 2021. It is now read-only.

fix(handler) ensure multivalueheaders are handled #59

Merged
merged 1 commit into from
Jun 15, 2021

Conversation

gszr
Copy link
Member

@gszr gszr commented Jun 9, 2021

Ensure the multiValueHeaders field is handled in AWS Lambda proxy
integration responses.

Fix #53.

@gszr gszr force-pushed the fix/handle-multi-value-headers branch 2 times, most recently from 048a37f to b76d430 Compare June 10, 2021 16:52
@Tieske
Copy link
Member

Tieske commented Jun 14, 2021

@gszr updated the docker login, saw you copied it from the docker repo, but that one was bad. See also: Kong/docker-kong#475

This should fix it.

@gszr
Copy link
Member Author

gszr commented Jun 14, 2021

Thank you @Tieske!

@gszr
Copy link
Member Author

gszr commented Jun 14, 2021

Seems this is failing completely now @Tieske. Any ideas?

@Tieske
Copy link
Member

Tieske commented Jun 15, 2021

because I'm an idiot

Ensure the `multiValueHeaders` field is handled in AWS Lambda proxy
integration responses.

Fix #53.
@Tieske Tieske force-pushed the fix/handle-multi-value-headers branch from edc36a5 to eae7751 Compare June 15, 2021 08:17
@Tieske Tieske merged commit 1c7e7b0 into master Jun 15, 2021
@Tieske Tieske deleted the fix/handle-multi-value-headers branch June 15, 2021 08:19
@gszr
Copy link
Member Author

gszr commented Jun 15, 2021

Thanks for the help, @Tieske! 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

multiValueHeaders is not returned in response
2 participants