-
-
Notifications
You must be signed in to change notification settings - Fork 657
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
feat: Added support for alias to have multiple filter criteria same as function #585
feat: Added support for alias to have multiple filter criteria same as function #585
Conversation
…event source mappings, same as lambda function
4fd5f1f
to
8ebe70f
Compare
Hi @pdecat @antonbabenko, Could you please review/approve this PR to fix the alias event mapping issue and align it with existing documentation please ? |
dynamic "filter" { | ||
for_each = try(flatten([each.value.filter_criteria]), []) | ||
|
||
content { | ||
pattern = try(filter.value.pattern, null) | ||
} |
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.
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!
## [7.7.0](v7.6.0...v7.7.0) (2024-06-18) ### Features * Added support for alias to have multiple filter criteria same as function ([#585](#585)) ([6549ca1](6549ca1))
This PR is included in version 7.7.0 🎉 |
thank you very much! :D |
… event source mappings, same as lambda function
Description
I have added support in the alias submodule to have multiple event source mapping
filter_criteria
in line with the main lambda function.Motivation and Context
I have wasted an entire evening trying to make event source mapping filter_criteria work on the alias with
operation error Lambda: CreateEventSourceMapping, https response error StatusCode: 400, RequestID: 4d25b930-c830-49e2-bbc7-dbd4add503f7, InvalidParameterValueException: Invalid filter pattern definition
.This is based on the example found here
Breaking Changes
Alias will be in line with the existing documentation.
How Has This Been Tested?
examples/*
to demonstrate and validate my change(s)examples/*
projectspre-commit run -a
on my pull request