fix: Correctly Set Location on non-RDF Notifications

When the negotiated media-type for notifications is not RDF, the `location` property is correctly set to the `Location` header in the generated notifications. Express-PREP now handles this by default.
nodeSolidServer · Oct 24, 2024 · 44dc0bc · 44dc0bc
1 parent 2da77b0
commit 44dc0bc
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 8 deletions.
diff --git a/lib/handlers/notify.js b/lib/handlers/notify.js
@@ -76,9 +76,7 @@ function handler (req, res, next) {
               mediaType
             })}`
           } else {
-            return defaultNotification({
-              ...(res.method === 'POST') && { location: res.getHeader('Content-Location') }
-            })
+            return defaultNotification()
           }
         }
       })

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -77,7 +77,7 @@
     "express-accept-events": "^0.3.0",
     "express-handlebars": "^5.3.5",
     "express-negotiate-events": "^0.3.0",
-    "express-prep": "^0.6.2",
+    "express-prep": "^0.6.3",
     "express-session": "^1.18.0",
     "extend": "^3.0.2",
     "from2": "^2.3.0",