fix: Don't overwrite blobRequest event fields.

Also fix missing bucket reference for bucket references with permissions when adding notifications.
nitrictech · Apr 11, 2024 · 34fba02 · 34fba02
1 parent e8243c6
commit 34fba02
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 7 deletions.
diff --git a/src/api/storage/v1/storage.ts b/src/api/storage/v1/storage.ts
@@ -120,7 +120,7 @@ export class Bucket {
       notificationPrefixFilter,
       ...middleware
     );
-    return notification['start']();
+    return notification['start'](this);
   }
 }
 

diff --git a/src/resources/bucket.ts b/src/resources/bucket.ts
@@ -138,12 +138,6 @@ export class BucketNotification {
           responseMessage.setId(message.getId());
 
           try {
-            blobEventRequest.setBucketName(this.options.bucket);
-            const blobEvent = new BlobEvent();
-            blobEvent.setKey(this.options.notificationPrefixFilter);
-            blobEvent.setType(this.options.notificationType);
-            blobEventRequest.setBlobEvent(blobEvent);
-
             if (bucket) {
               const ctx = BlobEventContext.fromRequest(
                 blobEventRequest,