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 Deploy): Recognize LogicalResourceId
in stackPolicy
#10097
fix(AWS Deploy): Recognize LogicalResourceId
in stackPolicy
#10097
Conversation
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.
Thank you @jmonsma ! It indeed looks as important fix. I've just pointed one issue in proposed regex. Once it's addressed we should be good
lib/plugins/aws/provider.js
Outdated
@@ -578,6 +585,10 @@ class AwsProvider { | |||
additionalProperties: false, | |||
required: ['securityGroupIds', 'subnetIds'], | |||
}, | |||
awsLogicalResourceId: { | |||
type: 'string', | |||
pattern: '^[#A-Za-z0-9-_./]+[*]?', |
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.
I believe we need a pattern for full string (so with $
at the end), as otherwise we allow any string after matching string, and that will not make a valid resource id
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, completely overlooked this one. I fixed the pattern.
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.
Thank you @jmonsma!
Codecov Report
@@ Coverage Diff @@
## master #10097 +/- ##
=======================================
Coverage 85.39% 85.39%
=======================================
Files 334 334
Lines 13604 13604
=======================================
Hits 11617 11617
Misses 1987 1987
Continue to review full report at Codecov.
|
LogicalResourceId
in stackPolicy
According the Cloudformation documentation it should support single and wildcard logical ID's
Addresses #10095