This repository has been archived by the owner on Jan 2, 2025. It is now read-only.

Take Bucket Ref into account #947

Merged

ar90n merged 1 commit into ar90n:master from itsmepetrov:master

Aug 20, 2024

Contributor

itsmepetrov commented Aug 19, 2024 •

edited

Loading

In our project we specify buckets via Ref:

events:
  - s3:
      existing: true
      bucket: !Ref IndexBucket
      rules:
        - prefix: data/batch/

unfortunately it breaks getResourceForBucket function

current PR fixes that issue


          Take Bucket Ref into account

e9eb236

itsmepetrov mentioned this pull request

TypeError: bucketName.charAt is not a function #948

Open

ar90n merged commit 5053509 into ar90n:master

1 check passed

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet