Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Always add lambda.amazonaws.com id #17

Merged
merged 2 commits into from
Jun 7, 2022
Merged

Always add lambda.amazonaws.com id #17

merged 2 commits into from
Jun 7, 2022

Conversation

nitrocode
Copy link
Member

what

  • Always add lambda.amazonaws.com id

why

  • lambda@edge requires both edgelambda and lambda identifiers

references

@nitrocode nitrocode added the patch A minor, backward compatible change label Jun 7, 2022
@nitrocode nitrocode requested review from a team as code owners June 7, 2022 03:25
@nitrocode
Copy link
Member Author

/test all

@jamengual jamengual merged commit 47dc230 into main Jun 7, 2022
@jamengual jamengual deleted the lambda-edge branch June 7, 2022 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jamengual jamengual jamengual approved these changes

@aknysh aknysh aknysh approved these changes

@r351574nc3 r351574nc3 Awaiting requested review from r351574nc3 r351574nc3 is a code owner automatically assigned from cloudposse/contributors

@brcnblc brcnblc Awaiting requested review from brcnblc brcnblc is a code owner automatically assigned from cloudposse/contributors

Assignees
No one assigned
Labels
patch A minor, backward compatible change
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Incorrect assume_role policy when enabling lambda_at_edge
4 participants
@nitrocode @jamengual @aknysh @cloudpossebot