Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

(s3): L2 Construct for S3 Object Lambda #13675

Closed
2 tasks
NukaCody opened this issue Mar 18, 2021 · 2 comments · Fixed by #15833
Closed
2 tasks

(s3): L2 Construct for S3 Object Lambda #13675

NukaCody opened this issue Mar 18, 2021 · 2 comments · Fixed by #15833
Labels
@aws-cdk/aws-lambda Related to AWS Lambda @aws-cdk/aws-s3 Related to Amazon S3 effort/medium Medium work item – several days of effort feature/new-construct A request for a new L2 construct feature-request A feature should be added or improved. p2

Comments

@NukaCody
Copy link

L2 Construct for S3 Object Lambda

Use Case

To allow the ability to add custom logic in front of S3 Get Request

Proposed Solution

  • @aws-cdk/aws-lambda-event-sources
  • @aws-cdk/aws-lambda
  • @aws-cdk/aws-s3
const fn = new lambda.Function(this, 'MyFunction', {
  runtime: lambda.Runtime.NODEJS_12_X,
  handler: 'index.handler',
  code: lambda.Code.fromAsset(path.join(__dirname, 'lambda-handler')),
});

const bucket = new Bucket(this, 'MyEncryptedBucket', {
    encryption: BucketEncryption.KMS
    objectLambda: fn
});

Other

Does not seem to have CFN support yet, so may have to wait a little bit.

  • 👋 I may be able to implement this feature request
  • ⚠️ This feature might incur a breaking change

This is a 🚀 Feature Request

@NukaCody NukaCody added feature-request A feature should be added or improved. needs-triage This issue or PR still needs to be triaged. labels Mar 18, 2021
@github-actions github-actions bot added @aws-cdk/aws-lambda Related to AWS Lambda @aws-cdk/aws-s3 Related to Amazon S3 labels Mar 18, 2021
@iliapolo iliapolo added effort/medium Medium work item – several days of effort p2 and removed needs-triage This issue or PR still needs to be triaged. labels Mar 20, 2021
@ericzbeard ericzbeard added the feature/new-construct A request for a new L2 construct label Apr 1, 2021
@jansepke
Copy link
Contributor

@mergify mergify bot closed this as completed in #15833 Mar 4, 2022
mergify bot pushed a commit that referenced this issue Mar 4, 2022
This PR adds an L2 construct for the S3 Object Lambda.

To avoid a circular dependency, the construct lives outside of the aws-s3 package.

Fixes #13675

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
@github-actions
Copy link

github-actions bot commented Mar 4, 2022

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see.
If you need more assistance, please either tag a team member or open a new issue that references this one.
If you wish to keep having a conversation with other community members under this issue feel free to do so.

TheRealAmazonKendra pushed a commit to TheRealAmazonKendra/aws-cdk that referenced this issue Mar 11, 2022
…15833)

This PR adds an L2 construct for the S3 Object Lambda.

To avoid a circular dependency, the construct lives outside of the aws-s3 package.

Fixes aws#13675

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
@aws-cdk/aws-lambda Related to AWS Lambda @aws-cdk/aws-s3 Related to Amazon S3 effort/medium Medium work item – several days of effort feature/new-construct A request for a new L2 construct feature-request A feature should be added or improved. p2
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants