Add ETag to existing empty folder listings, skip ETag for non-existent folders

melvincarvalho · melvincarvalho · commit 2afc01400400 · 2026-03-10T07:48:19.000+01:00
Existing folders with no entries get their real ETag (or "empty" fallback).
Non-existent folders intentionally omit ETag to prevent RS clients from
getting stuck in a 304 loop that skips push logic.
diff --git a/src/remotestorage.js b/src/remotestorage.js
@@ -100,6 +100,7 @@ export async function remoteStoragePlugin (fastify, options = {}) {
     const info = await storage.stat(storagePath)
 
     // Non-existent folder → return empty listing (RS spec: clients expect 200 to start writing)
+    // No ETag — forces RS clients to process the folder each sync cycle (304 would skip push logic)
     if (!info && storagePath.endsWith('/')) {
       return reply
         .header('Content-Type', 'application/ld+json')
@@ -126,6 +127,7 @@ export async function remoteStoragePlugin (fastify, options = {}) {
       if (!entries) {
         return reply
           .header('Content-Type', 'application/ld+json')
+          .header('ETag', info.etag || '"empty"')
           .header('Cache-Control', 'no-cache')
           .send({
             '@context': 'http://remotestorage.io/spec/folder-description',
