-
Notifications
You must be signed in to change notification settings - Fork 102
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
Update APIGateway example to use AWSLambdaEvents #76
Conversation
@fabianfett looks good, but formatting issues. you can run the sanity script locally to see the error and fix it |
let multiValueHeaders: [String: [String]]? | ||
let body: String? | ||
let isBase64Encoded: Bool? | ||
} |
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.
😸
@@ -30,13 +30,14 @@ let package = Package( | |||
.product(name: "AWSLambdaRuntime", package: "swift-aws-lambda-runtime"), | |||
]), | |||
.target(name: "Benchmark", dependencies: [ | |||
.product(name: "AWSLambdaRuntime", package: "swift-aws-lambda-runtime"), | |||
.product(name: "AWSLambdaRuntimeCore", package: "swift-aws-lambda-runtime"), |
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.
👍
@tomerd let me test this once on AWS before merging ;) |
2c99fd5
to
a3759f5
Compare
a3759f5
to
23b3971
Compare
@drexin Do you know whether we will get the static-stdlib linking in without the linker flags in |
/usr/lib/swift/linux/libswiftCore.so \ | ||
/usr/lib/swift/linux/libswiftDispatch.so \ | ||
/usr/lib/swift/linux/libswiftGlibc.so \ | ||
"$target" |
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.
@fabianfett can we rollback this one change for now? I would like to work on eliminating ICU for lambda then we can do a separate PR for that
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.
@tomerd done!
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.
lgtm, one comment to rollback unrelated change we need to address separately
bf9e88b
to
9b61a2e
Compare
Motivation
We now have an APIGateway request and response on board. Use it in the Sample.
Changes