Closed nsarychev closed 4 years ago
Issue #, if available:
Description of changes: This commit https://github.com/awslabs/aws-lambda-go-api-proxy/commit/d7b2b4e73251828d55ff525983683f6e81803866 removed setting single-value headers, which caused all of our tests against gin break. This PR adds back the code to populate single-value headers as it did before. Please consider this for your review.
Note: There are commits in history from my previous merged PR https://github.com/awslabs/aws-lambda-go-api-proxy/pull/33. I was using same fork just updated it.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
@sapessi Could you please review this when you get a chance? Thank you!
As soon as the CI for the merge completes I'll tag v0.6.0
v0.6.0
Thank you for such a quick response @sapessi!
Issue #, if available:
Description of changes: This commit https://github.com/awslabs/aws-lambda-go-api-proxy/commit/d7b2b4e73251828d55ff525983683f6e81803866 removed setting single-value headers, which caused all of our tests against gin break. This PR adds back the code to populate single-value headers as it did before. Please consider this for your review.
Note: There are commits in history from my previous merged PR https://github.com/awslabs/aws-lambda-go-api-proxy/pull/33. I was using same fork just updated it.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.