Variables: Support resolving "raw" JSON string form of SSM params #9229
Conversation
Codecov Report
@@ Coverage Diff @@
## master #9229 +/- ##
=======================================
Coverage 86.94% 86.94%
=======================================
Files 315 315
Lines 11672 11678 +6
=======================================
+ Hits 10148 10154 +6
Misses 1524 1524
Continue to review full report at Codecov.
|
| @@ -20,6 +20,8 @@ module.exports = (serverlessInstance) => { | |||
| Error: ServerlessError, | |||
| errorMessage: 'Non-string address argument in variable "ssm" source: %v', | |||
| }); | |||
| const region = !params || !params[0] || params[0] === 'raw' ? null : params[0]; | |||
There was a problem hiding this comment.
Looks like this change is causing an error for me:
Cannot resolve serverless.yml: Variables resolution errored with:
- Cannot resolve variable at "<var name>": Missing region in config
I think this logic is resulting in region = null, but it used to resolve to false?
I haven't dug much deeper, but I was able to work around the error by adding the region param, ie. ssm(us-east-1):...
Closes: #9217