feat: hash objects for etag value

server/api/v1/object/[id]/index.get.ts

@@ -12,15 +12,16 @@ export default defineEventHandler(async (h3) => {
     throw createError({ statusCode: 404, statusMessage: "Object not found" });
 
   // https://developer.mozilla.org/en-US/docs/Web/HTTP/Reference/Headers/ETag
-  const etagValue = h3.headers.get("If-None-Match");
+  const etagRequestValue = h3.headers.get("If-None-Match");
-  if (etagValue !== null) {
+  const etagActualValue = await objectHandler.fetchHash(id);
+  if (etagRequestValue !== null && etagActualValue === etagRequestValue) {
     // would compare if etag is valid, but objects should never change
     setResponseStatus(h3, 304);
     return null;
   }
 
-  // just return object id has etag since object should never change
+  // TODO: fix undefined etagValue
-  setHeader(h3, "ETag", id);
+  setHeader(h3, "ETag", etagActualValue ?? "");
   setHeader(h3, "Content-Type", object.mime);
   setHeader(
     h3,

server/api/v1/object/[id]/index.head.ts

@@ -13,8 +13,9 @@ export default defineEventHandler(async (h3) => {
     throw createError({ statusCode: 404, statusMessage: "Object not found" });
 
   // https://developer.mozilla.org/en-US/docs/Web/HTTP/Reference/Headers/ETag
-  const etagValue = h3.headers.get("If-None-Match");
+  const etagRequestValue = h3.headers.get("If-None-Match");
-  if (etagValue !== null) {
+  const etagActualValue = await objectHandler.fetchHash(id);
+  if (etagRequestValue !== null && etagActualValue === etagRequestValue) {
     // would compare if etag is valid, but objects should never change
     setResponseStatus(h3, 304);
     return null;

server/internal/objects/fsBackend.ts

@@ -7,15 +7,22 @@ import {
 } from "./objectHandler";
 
 import sanitize from "sanitize-filename";
-
+import { LRUCache } from "lru-cache";
 import fs from "fs";
 import path from "path";
 import { Readable, Stream } from "stream";
+import { createHash } from "crypto";
 
 export class FsObjectBackend extends ObjectBackend {
   private baseObjectPath: string;
   private baseMetadataPath: string;
 
+  // TODO: should probably make this save into db or something if we agree to never
+  // overwrite an object
+  private cache = new LRUCache<string, string>({
+    max: 1000, // number of items
+  });
+
   constructor() {
     super();
     const basePath = process.env.FS_BACKEND_PATH ?? "./.data/objects";
@@ -35,6 +42,9 @@ export class FsObjectBackend extends ObjectBackend {
     const objectPath = path.join(this.baseObjectPath, sanitize(id));
     if (!fs.existsSync(objectPath)) return false;
 
+    // remove item from cache
+    this.cache.delete(id);
+
     if (source instanceof Readable) {
       const outputStream = fs.createWriteStream(objectPath);
       source.pipe(outputStream, { end: true });
@@ -52,7 +62,8 @@ export class FsObjectBackend extends ObjectBackend {
   async startWriteStream(id: ObjectReference) {
     const objectPath = path.join(this.baseObjectPath, sanitize(id));
     if (!fs.existsSync(objectPath)) return undefined;
-
+    // remove item from cache
+    this.cache.delete(id);
     return fs.createWriteStream(objectPath);
   }
   async create(
@@ -100,6 +111,8 @@ export class FsObjectBackend extends ObjectBackend {
     const objectPath = path.join(this.baseObjectPath, sanitize(id));
     if (!fs.existsSync(objectPath)) return true;
     fs.rmSync(objectPath);
+    // remove item from cache
+    this.cache.delete(id);
     return true;
   }
   async fetchMetadata(
@@ -125,4 +138,26 @@ export class FsObjectBackend extends ObjectBackend {
     fs.writeFileSync(metadataPath, JSON.stringify(metadata));
     return true;
   }
+  async fetchHash(id: ObjectReference): Promise<string | undefined> {
+    const cacheResult = this.cache.get(id);
+    if (cacheResult !== undefined) return cacheResult;
+
+    const obj = await this.fetch(id);
+    if (obj === undefined) return;
+
+    // local variable to point to object
+    const cache = this.cache;
+
+    // hash object
+    const hash = createHash("md5");
+    hash.setEncoding("hex");
+    obj.on("end", function () {
+      hash.end();
+      cache.set(id, hash.read());
+    });
+    // read obj into hash
+    obj.pipe(hash);
+
+    return this.cache.get(id);
+  }
 }

server/internal/objects/objectHandler.ts

@@ -49,20 +49,21 @@ export abstract class ObjectBackend {
   abstract create(
     id: string,
     source: Source,
-    metadata: ObjectMetadata
+    metadata: ObjectMetadata,
   ): Promise<ObjectReference | undefined>;
   abstract createWithWriteStream(
     id: string,
-    metadata: ObjectMetadata
+    metadata: ObjectMetadata,
   ): Promise<Writable | undefined>;
   abstract delete(id: ObjectReference): Promise<boolean>;
   abstract fetchMetadata(
-    id: ObjectReference
+    id: ObjectReference,
   ): Promise<ObjectMetadata | undefined>;
   abstract writeMetadata(
     id: ObjectReference,
-    metadata: ObjectMetadata
+    metadata: ObjectMetadata,
   ): Promise<boolean>;
+  abstract fetchHash(id: ObjectReference): Promise<string | undefined>;
 
   private async fetchMimeType(source: Source) {
     if (source instanceof ReadableStream) {
@@ -86,7 +87,7 @@ export abstract class ObjectBackend {
     id: string,
     sourceFetcher: () => Promise<Source>,
     metadata: { [key: string]: string },
-    permissions: Array<string>
+    permissions: Array<string>,
   ) {
     const { source, mime } = await this.fetchMimeType(await sourceFetcher());
     if (!mime)
@@ -102,7 +103,7 @@ export abstract class ObjectBackend {
   async createWithStream(
     id: string,
     metadata: { [key: string]: string },
-    permissions: Array<string>
+    permissions: Array<string>,
   ) {
     return this.createWithWriteStream(id, {
       permissions,
@@ -111,6 +112,12 @@ export abstract class ObjectBackend {
     });
   }
 
+  /**
+   * Fetches object, but also checks if user has perms to access it 
+   * @param id 
+   * @param userId 
+   * @returns 
+   */
   async fetchWithPermissions(id: ObjectReference, userId?: string) {
     const metadata = await this.fetchMetadata(id);
     if (!metadata) return;
@@ -147,7 +154,7 @@ export abstract class ObjectBackend {
   async writeWithPermissions(
     id: ObjectReference,
     sourceFetcher: () => Promise<Source>,
-    userId?: string
+    userId?: string,
   ) {
     const metadata = await this.fetchMetadata(id);
     if (!metadata) return false;