PivotResponseWriter

[michielbdejong]

Display containers even if trailing slash is missing from the URL

[michielbdejong]

I was able to wrap some own code around the http sequence handler.
Its output is undefined, but it calls methods on input.response.
I can see input.response.statusCode, and this would be the perfect place to catch 401/403/404 and rewrite to 301 if the resource exists after toggling the switch, except that when I try to call input.res.setHeader, it errors with:

Error [ERR_HTTP_HEADERS_SENT]: Cannot set headers after they are sent to the client

[michielbdejong]

I'm re-adding the code from #19 to see where that gets us:

-    // In the future we want to use getNormalizedMetadata and redirect in case the identifier differs
-    let metadata = await this.accessor.getMetadata(identifier);
+    // We use getNormalizedMetadata and redirect in case the identifier differs
+    let metadata = await this.getNormalizedMetadata(identifier);
+    // Redirect in case the identifier differs
+    if (metadata.identifier.value !== identifier.path) {
+      throw new RedirectHttpError(302, 'Found', metadata.identifier.value);
+    }

Actually while re-reading this I don't understand why it would lead to locking problems.

[michielbdejong]

Maybe I can hook into the BasicResponseWriter, here

[michielbdejong]

Merging this so we can tag a release and test it.