@constructive-io/bucket-provisioner 0.2.0 → 0.2.1

package/esm/provisioner.d.ts

@@ -85,6 +85,10 @@ export declare class BucketProvisioner {
     bucketExists(bucketName: string): Promise<boolean>;
     /**
      * Configure S3 Block Public Access settings.
+     *
+     * Gracefully skips if the S3-compatible backend (e.g. MinIO) does not
+     * support PutPublicAccessBlock — the operation is best-effort since
+     * not all providers implement this AWS-specific API.
      */
     setPublicAccessBlock(bucketName: string, config: PublicAccessBlockConfig): Promise<void>;
     /**
@@ -93,10 +97,17 @@ export declare class BucketProvisioner {
     setBucketPolicy(bucketName: string, policy: BucketPolicyDocument): Promise<void>;
     /**
      * Delete an S3 bucket policy (used to clear leftover public policies).
+     *
+     * Gracefully handles backends that don't support this operation or have
+     * no policy to delete.
      */
     deleteBucketPolicy(bucketName: string): Promise<void>;
     /**
      * Set CORS configuration on an S3 bucket.
+     *
+     * Gracefully skips if the S3-compatible backend (e.g. older MinIO) does
+     * not support PutBucketCors — CORS is best-effort since not all providers
+     * implement this API via the same endpoint path.
      */
     setCors(bucketName: string, rules: CorsRule[]): Promise<void>;
     /**

package/esm/provisioner.js

@@ -168,6 +168,10 @@ export class BucketProvisioner {
     }
     /**
      * Configure S3 Block Public Access settings.
+     *
+     * Gracefully skips if the S3-compatible backend (e.g. MinIO) does not
+     * support PutPublicAccessBlock — the operation is best-effort since
+     * not all providers implement this AWS-specific API.
      */
     async setPublicAccessBlock(bucketName, config) {
         try {
@@ -177,6 +181,18 @@ export class BucketProvisioner {
             }));
         }
         catch (err) {
+            // Some S3-compatible backends (e.g. older MinIO) don't support
+            // PutPublicAccessBlock. Treat XML parse errors or "not implemented"
+            // responses as non-fatal — the bucket is still usable.
+            if (err.Code === 'XmlParseException' ||
+                err.name === 'XmlParseException' ||
+                err.Code === 'NotImplemented' ||
+                err.name === 'NotImplemented' ||
+                err.message?.includes('not well-formed') ||
+                err.message?.includes('not implemented') ||
+                err.message?.includes('PublicAccessBlockConfiguration')) {
+                return;
+            }
             throw new ProvisionerError('POLICY_FAILED', `Failed to set public access block on '${bucketName}': ${err.message}`, err);
         }
     }
@@ -196,6 +212,9 @@ export class BucketProvisioner {
     }
     /**
      * Delete an S3 bucket policy (used to clear leftover public policies).
+     *
+     * Gracefully handles backends that don't support this operation or have
+     * no policy to delete.
      */
     async deleteBucketPolicy(bucketName) {
         try {
@@ -203,7 +222,14 @@ export class BucketProvisioner {
         }
         catch (err) {
             // No policy to delete — that's fine
-            if (err.name === 'NoSuchBucketPolicy' || err.$metadata?.httpStatusCode === 404) {
+            if (err.name === 'NoSuchBucketPolicy' ||
+                err.$metadata?.httpStatusCode === 404 ||
+                err.Code === 'XmlParseException' ||
+                err.name === 'XmlParseException' ||
+                err.Code === 'NotImplemented' ||
+                err.name === 'NotImplemented' ||
+                err.message?.includes('not well-formed') ||
+                err.message?.includes('not implemented')) {
                 return;
             }
             throw new ProvisionerError('POLICY_FAILED', `Failed to delete bucket policy on '${bucketName}': ${err.message}`, err);
@@ -211,6 +237,10 @@ export class BucketProvisioner {
     }
     /**
      * Set CORS configuration on an S3 bucket.
+     *
+     * Gracefully skips if the S3-compatible backend (e.g. older MinIO) does
+     * not support PutBucketCors — CORS is best-effort since not all providers
+     * implement this API via the same endpoint path.
      */
     async setCors(bucketName, rules) {
         try {
@@ -228,6 +258,18 @@ export class BucketProvisioner {
             }));
         }
         catch (err) {
+            // Some S3-compatible backends (e.g. older MinIO) don't support
+            // PutBucketCors via the standard path. Treat XML parse errors or
+            // "not implemented" responses as non-fatal.
+            if (err.Code === 'XmlParseException' ||
+                err.name === 'XmlParseException' ||
+                err.Code === 'NotImplemented' ||
+                err.name === 'NotImplemented' ||
+                err.message?.includes('not well-formed') ||
+                err.message?.includes('not implemented') ||
+                err.message?.includes('CORSConfiguration')) {
+                return;
+            }
             throw new ProvisionerError('CORS_FAILED', `Failed to set CORS on '${bucketName}': ${err.message}`, err);
         }
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@constructive-io/bucket-provisioner",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "author": "Constructive <developers@constructive.io>",
   "description": "S3-compatible bucket provisioning library — create buckets, configure privacy policies, CORS, and access controls",
   "main": "index.js",
@@ -43,5 +43,5 @@
     "storage",
     "constructive"
   ],
-  "gitHead": "fe60f7b81252eea53dce227bb581d5ae2ef0ec36"
+  "gitHead": "79cd3e66871804a22c672c7ca2fa5e2105d4b368"
 }

package/provisioner.d.ts

@@ -85,6 +85,10 @@ export declare class BucketProvisioner {
     bucketExists(bucketName: string): Promise<boolean>;
     /**
      * Configure S3 Block Public Access settings.
+     *
+     * Gracefully skips if the S3-compatible backend (e.g. MinIO) does not
+     * support PutPublicAccessBlock — the operation is best-effort since
+     * not all providers implement this AWS-specific API.
      */
     setPublicAccessBlock(bucketName: string, config: PublicAccessBlockConfig): Promise<void>;
     /**
@@ -93,10 +97,17 @@ export declare class BucketProvisioner {
     setBucketPolicy(bucketName: string, policy: BucketPolicyDocument): Promise<void>;
     /**
      * Delete an S3 bucket policy (used to clear leftover public policies).
+     *
+     * Gracefully handles backends that don't support this operation or have
+     * no policy to delete.
      */
     deleteBucketPolicy(bucketName: string): Promise<void>;
     /**
      * Set CORS configuration on an S3 bucket.
+     *
+     * Gracefully skips if the S3-compatible backend (e.g. older MinIO) does
+     * not support PutBucketCors — CORS is best-effort since not all providers
+     * implement this API via the same endpoint path.
      */
     setCors(bucketName: string, rules: CorsRule[]): Promise<void>;
     /**

package/provisioner.js

@@ -171,6 +171,10 @@ class BucketProvisioner {
     }
     /**
      * Configure S3 Block Public Access settings.
+     *
+     * Gracefully skips if the S3-compatible backend (e.g. MinIO) does not
+     * support PutPublicAccessBlock — the operation is best-effort since
+     * not all providers implement this AWS-specific API.
      */
     async setPublicAccessBlock(bucketName, config) {
         try {
@@ -180,6 +184,18 @@ class BucketProvisioner {
             }));
         }
         catch (err) {
+            // Some S3-compatible backends (e.g. older MinIO) don't support
+            // PutPublicAccessBlock. Treat XML parse errors or "not implemented"
+            // responses as non-fatal — the bucket is still usable.
+            if (err.Code === 'XmlParseException' ||
+                err.name === 'XmlParseException' ||
+                err.Code === 'NotImplemented' ||
+                err.name === 'NotImplemented' ||
+                err.message?.includes('not well-formed') ||
+                err.message?.includes('not implemented') ||
+                err.message?.includes('PublicAccessBlockConfiguration')) {
+                return;
+            }
             throw new types_1.ProvisionerError('POLICY_FAILED', `Failed to set public access block on '${bucketName}': ${err.message}`, err);
         }
     }
@@ -199,6 +215,9 @@ class BucketProvisioner {
     }
     /**
      * Delete an S3 bucket policy (used to clear leftover public policies).
+     *
+     * Gracefully handles backends that don't support this operation or have
+     * no policy to delete.
      */
     async deleteBucketPolicy(bucketName) {
         try {
@@ -206,7 +225,14 @@ class BucketProvisioner {
         }
         catch (err) {
             // No policy to delete — that's fine
-            if (err.name === 'NoSuchBucketPolicy' || err.$metadata?.httpStatusCode === 404) {
+            if (err.name === 'NoSuchBucketPolicy' ||
+                err.$metadata?.httpStatusCode === 404 ||
+                err.Code === 'XmlParseException' ||
+                err.name === 'XmlParseException' ||
+                err.Code === 'NotImplemented' ||
+                err.name === 'NotImplemented' ||
+                err.message?.includes('not well-formed') ||
+                err.message?.includes('not implemented')) {
                 return;
             }
             throw new types_1.ProvisionerError('POLICY_FAILED', `Failed to delete bucket policy on '${bucketName}': ${err.message}`, err);
@@ -214,6 +240,10 @@ class BucketProvisioner {
     }
     /**
      * Set CORS configuration on an S3 bucket.
+     *
+     * Gracefully skips if the S3-compatible backend (e.g. older MinIO) does
+     * not support PutBucketCors — CORS is best-effort since not all providers
+     * implement this API via the same endpoint path.
      */
     async setCors(bucketName, rules) {
         try {
@@ -231,6 +261,18 @@ class BucketProvisioner {
             }));
         }
         catch (err) {
+            // Some S3-compatible backends (e.g. older MinIO) don't support
+            // PutBucketCors via the standard path. Treat XML parse errors or
+            // "not implemented" responses as non-fatal.
+            if (err.Code === 'XmlParseException' ||
+                err.name === 'XmlParseException' ||
+                err.Code === 'NotImplemented' ||
+                err.name === 'NotImplemented' ||
+                err.message?.includes('not well-formed') ||
+                err.message?.includes('not implemented') ||
+                err.message?.includes('CORSConfiguration')) {
+                return;
+            }
             throw new types_1.ProvisionerError('CORS_FAILED', `Failed to set CORS on '${bucketName}': ${err.message}`, err);
         }
     }