Skip to content
This repository has been archived by the owner on May 16, 2023. It is now read-only.

Make ApiGatewayV2httpRequestContext#authentication optional. #54

Merged
merged 1 commit into from
Feb 5, 2022
Merged

Make ApiGatewayV2httpRequestContext#authentication optional. #54

merged 1 commit into from
Feb 5, 2022

Conversation

calavera
Copy link
Owner

Fixes #46

Signed-off-by: David Calavera [email protected]

@LegNeato
Copy link
Contributor

I don't know much about this event...should it be fixed in go and then we can pick it up rather than hardcoding it here?

https://github.com/aws/aws-lambda-go/blob/main/events/apigw.go#L79

This project keys off of omitempty and it being a pointer.

@calavera
Copy link
Owner Author

calavera commented Feb 5, 2022

I opened a PR for the Go team to fix it aws/aws-lambda-go#426

I'm going to merge this change because I'd like to unblock people reporting this issue in #46.

@calavera calavera merged commit 32a3939 into calavera:master Feb 5, 2022
@calavera calavera deleted the optional_authentication branch February 5, 2022 21:14
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.

Authentication field in ApiGatewayV2httpRequestContext should be optional
2 participants