fix: Correctly Set Location on non-RDF Notifications

CxRes · CxRes · commit 44dc0bce7816 · 2024-10-24T05:47:17.000+05:30
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.
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",

Original file line number	Diff line number	Diff line change
`@@ -76,9 +76,7 @@ function handler (req, res, next) {`
`76`	`76`	`mediaType`
`77`	`77`	})}`
`78`	`78`	`} else {`
`79`		`- return defaultNotification({`
`80`		`- ...(res.method === 'POST') && { location: res.getHeader('Content-Location') }`
`81`		`- })`
	`79`	`+ return defaultNotification()`
`82`	`80`	`}`
`83`	`81`	`}`
`84`	`82`	`})`