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(AWS S3): Recognize ExpirationInDays
property for s3
events
#10083
Conversation
@medikoo I'm not sure how to assign a reviewer to this specifically, but since you added the comment to the initial issue, would you mind reviewing the PR for it? |
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 @ROSeaboyer !
To be able to address similar issues easier in a future, It'll be great to create a reusable util for that. So we can use for such properties:
ExpirationInDays: cfValue({ type: 'integer', minimum: 0 })
I propose to create lib/utils/aws-schema-get-cf-value.js
util, which will return something as:
anyOf: [
value,
{ $ref: '#/definitions/awsCfFunction' },
{ $ref: '#/definitions/awsCfIf' },
]
@pgrzesik what do you think?
@medikoo I just made that change; can you let me know what you think? |
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.
@ROSeaboyer thanks for update. Looks good to me!
@pgrzesik what are your thoughts?
Thanks @ROSeaboyer and sorry @medikoo - I'm missed previous notifcation. I think it looks good, we will definitely have more similar cases and that util will come in handy 👍 |
Codecov Report
@@ Coverage Diff @@
## master #10083 +/- ##
=======================================
Coverage 85.41% 85.41%
=======================================
Files 333 334 +1
Lines 13586 13589 +3
=======================================
+ Hits 11604 11607 +3
Misses 1982 1982
Continue to review full report at Codecov.
|
ExpirationInDays
property for s3
events
Closes: #10081