@michaelthielemann/kestrel-blobstore-s3 5.0.3 → 5.1.0

package/README.md

@@ -4,8 +4,9 @@ via `@aws-sdk/client-s3`. Config: `bucket`, optional `prefix` (`"site/"`), `regi
 `forcePathStyle`, `accessKeyId`/`secretAccessKey` (omit to use the SDK's default credential chain:
 environment, profile, instance role), `timeoutMs` (default 10000, used for both the connection and
 the request timeout, so a silent socket never hangs a pipeline step) and `maxAttempts` (default 3,
-the SDK's retry limit including the first try). `list()` reports `application/octet-stream` as
-content type because S3 listings carry no metadata; `get()` returns the stored type.
+the SDK's retry limit including the first try). `put` stores the `contentType` hint as the object's
+`Content-Type`, so a bucket served directly answers with the right type; `get` returns bytes only
+and `list` reports key and size. The store never wrote sidecar files.
 
 `pnpm test:s3` runs the `blobstore@1` contract test against a real MinIO instance in Podman
 (starts it, creates the bucket, runs `minio.test.ts`, always stops the container).

package/dist/impl.js

@@ -50,9 +50,9 @@ export function createBlobstoreS3(config, client = createClient(config)) {
             if (client instanceof S3Client)
                 client.destroy();
         },
-        async put(key, blob) {
+        async put(key, data, options = {}) {
             try {
-                await client.send(new PutObjectCommand({ Bucket: config.bucket, Key: fullKey(key), Body: blob.data, ContentType: blob.contentType }));
+                await client.send(new PutObjectCommand({ Bucket: config.bucket, Key: fullKey(key), Body: data, ...(options.contentType === undefined ? {} : { ContentType: options.contentType }) }));
                 return ok();
             }
             catch (cause) {
@@ -66,7 +66,7 @@ export function createBlobstoreS3(config, client = createClient(config)) {
                 const out = (await client.send(new GetObjectCommand({ Bucket: config.bucket, Key: fullKey(key) })));
                 if (!out.Body)
                     return ok(null);
-                return ok({ data: await out.Body.transformToByteArray(), contentType: out.ContentType ?? "application/octet-stream" });
+                return ok(await out.Body.transformToByteArray());
             }
             catch (cause) {
                 if (cause instanceof NoSuchKey || cause.name === "NoSuchKey")
@@ -121,7 +121,7 @@ export function createBlobstoreS3(config, client = createClient(config)) {
                     for (const item of page.Contents ?? []) {
                         if (item.Key === undefined)
                             continue;
-                        out.push({ key: item.Key.slice(config.prefix.length), size: item.Size ?? 0, contentType: "application/octet-stream" });
+                        out.push({ key: item.Key.slice(config.prefix.length), size: item.Size ?? 0 });
                     }
                     token = page.NextContinuationToken;
                 } while (token);