feat: Set MS-Author-Via header.

Closes #539
CommunitySolidServer · Jan 26, 2021 · 8c2f737 · 8c2f737
1 parent fe3957f
commit 8c2f737
Show file tree

Hide file tree

Showing 4 changed files with 23 additions and 0 deletions.
diff --git a/config/presets/ldp/response-writer.json b/config/presets/ldp/response-writer.json
@@ -5,6 +5,15 @@
       "@id": "urn:solid-server:default:MetadataSerializer",
       "@type": "SequenceHandler",
       "SequenceHandler:_handlers": [
+        {
+          "@type": "ConstantMetadataWriter",
+          "ConstantMetadataWriter:_headers": [
+            {
+              "ConstantMetadataWriter:_headers_key": "MS-Author-Via",
+              "ConstantMetadataWriter:_headers_value": "SPARQL"
+            }
+          ]
+        },
         {
           "@type": "MappedMetadataWriter",
           "MappedMetadataWriter:_headerMap": [

diff --git a/config/presets/middleware.json b/config/presets/middleware.json
@@ -19,6 +19,7 @@
           "CorsHandler:_options_credentials": true,
           "CorsHandler:_options_exposedHeaders": [
             "Location",
+            "MS-Author-Via",
             "Updates-Via"
           ]
         },

diff --git a/test/integration/LdpHandlerWithoutAuth.test.ts b/test/integration/LdpHandlerWithoutAuth.test.ts
@@ -81,6 +81,7 @@ describe.each(stores)('An LDP handler without auth using %s', (name, { storeUrn,
     expect(response.statusCode).toBe(200);
     expect(response._getBuffer().toString()).toContain('TESTFILE0');
     expect(response.getHeaders().link).toBe(`<${LDP.Resource}>; rel="type"`);
+    expect(response.getHeaders()['ms-author-via']).toBe('SPARQL');
 
     // DELETE
     await resourceHelper.deleteResource(id);

diff --git a/test/integration/Middleware.test.ts b/test/integration/Middleware.test.ts
@@ -92,6 +92,18 @@ describe('An Express server with middleware', (): void => {
     expect(exposed.split(/\s*,\s*/u)).toContain('Location');
   });
 
+  it('exposes the MS-Author-Via header via CORS.', async(): Promise<void> => {
+    const res = await request(server).get('/').expect(200);
+    const exposed = res.header['access-control-expose-headers'];
+    expect(exposed.split(/\s*,\s*/u)).toContain('MS-Author-Via');
+  });
+
+  it('exposes the Updates-Via header via CORS.', async(): Promise<void> => {
+    const res = await request(server).get('/').expect(200);
+    const exposed = res.header['access-control-expose-headers'];
+    expect(exposed.split(/\s*,\s*/u)).toContain('Updates-Via');
+  });
+
   it('sends incoming requests to the handler.', async(): Promise<void> => {
     const response = request(server).get('/').set('Host', 'test.com');
     expect(response).toBeDefined();