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
fix(aws-lambda,netlify-lambda): binary body v2 and cookies v1 #1683
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1683 +/- ##
=======================================
Coverage 77.79% 77.79%
=======================================
Files 76 76
Lines 7836 7836
Branches 805 805
=======================================
Hits 6096 6096
Misses 1738 1738
Partials 2 2 |
body: awsBody.body, | ||
isBase64Encoded: awsBody.type === "binary", | ||
...(cookies.length > 0 && { | ||
multiValueHeaders: { "set-cookie": cookies }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just checking, was this (another) fix or refactor to avoid empty object?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Refactor due to the initial fix, I made the modification here too so that both aws-lambda
and netlify-lambda
have consistent behaviour (Technically it improves the performances if its empty but not in a meaningful way...)
@Hebilicious Since planning for next patch release, what is blocker for this PR? Do you need to verify it against a deployment? |
Was just in draft because WIP, it should be good to go now ππ½ |
It would be great if this is merged. I'm having issues with hosting IPX on my lambda function. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks (i haven't test it through but i trust you did)
π Linked issue
Fix #852
β Type of change
π Description
The aws-lambda preset works with apigw v1 and v2.
This PR fix the body type for apigw v2, and the cookies for apigw v1
π Checklist