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.

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

Confusing Error message for defining S3 event #109

Closed
rowanu opened this issue Mar 17, 2017 · 1 comment
Closed

Confusing Error message for defining S3 event #109

rowanu opened this issue Mar 17, 2017 · 1 comment

Comments

@rowanu
Copy link
Contributor

rowanu commented Mar 17, 2017

Was getting this error message on a deploy command:

Failed to create the changeset: Waiter ChangeSetCreateComplete failed: Waiter encountered a terminal failure state Status: FAILED. Reason: Transform AWS::Serverless-2016-10-31 failed with: Invalid Serverless Application Specification document. Number of errors found: 1. Resource with id [ResourceFile] is invalid. property Event not defined for resource of type S3

The text makes it sound like I'm missing the Event property, but the property I was missing was Events. Changing my incorrect property name made the error go away.

@sanathkr sanathkr added the bug label Jun 28, 2017
@sanathkr
Copy link
Contributor

The error message is complaining that you specified a property called Event that it doesn't recognize. This is indeed confusing given the error message. I have made the required code change. Will get it deployed in the next release.

Closing this bug for now because the change is made.

hawflau added a commit that referenced this issue Apr 6, 2022
* feat: Support for Lambda URL (#86)

* feat: changed CorsConfig property to Cors (#89)

* feat: add list of "unreleased" properties (#91)

This change hides the unreleased properties from the "Globals" error message

* Lambda URL integ test (#94)

* remove functionUrlConfig from the unrelease properties (#97)

* remove functionUrlConfig from the unrelease properties

* updates

* Function Url new changes  (#109)

* update to Function Url

* first update

* reverted to support autopublishing and added validation for AuthType values

* adding Integration test

* fix: fix paths IF intrinsic bug (#107)

* fix: fix paths IF intrinsic bug

* missed bit of code

* add NoValue tests

* review feedback and refactoring

* naming fix

* review feedback naming and comments

* add missing test + add extra check for api key on tests

* integration test updated (#122)

Co-authored-by: jonife <79116465+jonife@users.noreply.github.com>
Co-authored-by: Renato Valenzuela <37676028+valerena@users.noreply.github.com>
Co-authored-by: Ruperto Torres <86501267+torresxb1@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants