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

add basic support for QueueConfigurations #598

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

80avin
Copy link

@80avin 80avin commented Sep 1, 2022

Add preliminary support for NotificationConfiguration ( only QueueConfigurations yet )

Allows sending SQS events on S3 events.

fixes #98

@ar90n
Copy link
Owner

ar90n commented Dec 12, 2022

@80avin
Thanks for your PR. But I couldn't merge this PR for serverless-s3-local.
Because I want to keep serverless-s3-local simple. This means that serverless-s3-local isn't designed to support the full features of S3. If you want full-feature supported S3 integrated with lambda, Localstack will be helpful for your purpose.

@80avin
Copy link
Author

80avin commented Dec 12, 2022

@ar90n I understand.
It's okay if you want to delete or modify this PR. I created it since I had to implement this feature for my requirements, and then thought of pushing for more visibility.

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

Successfully merging this pull request may close these issues.

Unable to use event put-bucket-notification-configuration
2 participants