serverless-heaven / serverless-aws-alias

Alias support for Serverless 1.x
MIT License
189 stars 68 forks source link

Fixed detection of Principal for Serverless 1.27 #121

Closed HyperBrain closed 6 years ago

HyperBrain commented 6 years ago

Fixes #120

Serverless 1.27 changed the construction of the Principal propertiy in permission resources from a string to a Fn::Join. This breaks the principal detection that sorts the permissions by services for the alias stack.

I extended the principal validation to adhere to these changes. All permission objects (APIG, CWEvents and SNS) should now be attached to the correct function aliases again.

To verify the fix, check if the permissions show up in the AWS Lambda console and are attached only to the aliases, not to $LATEST.