feat: Set Accept-Patch header.

CommunitySolidServer · Jan 26, 2021 · 153d2d9 · 153d2d9
1 parent 394e28b
commit 153d2d9
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 0 deletions.
diff --git a/config/presets/ldp/response-writer.json b/config/presets/ldp/response-writer.json
@@ -8,6 +8,10 @@
         {
           "@type": "ConstantMetadataWriter",
           "ConstantMetadataWriter:_headers": [
+            {
+              "ConstantMetadataWriter:_headers_key": "Accept-Patch",
+              "ConstantMetadataWriter:_headers_value": "application/sparql-update"
+            },
             {
               "ConstantMetadataWriter:_headers_key": "MS-Author-Via",
               "ConstantMetadataWriter:_headers_value": "SPARQL"

diff --git a/config/presets/middleware.json b/config/presets/middleware.json
@@ -18,6 +18,7 @@
           ],
           "CorsHandler:_options_credentials": true,
           "CorsHandler:_options_exposedHeaders": [
+            "Accept-Patch",
             "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()['accept-patch']).toBe('application/sparql-update');
     expect(response.getHeaders()['ms-author-via']).toBe('SPARQL');
 
     // DELETE

diff --git a/test/integration/Middleware.test.ts b/test/integration/Middleware.test.ts
@@ -86,6 +86,12 @@ describe('An Express server with middleware', (): void => {
     expect(res.header).toEqual(expect.objectContaining({ 'access-control-allow-origin': 'test.com' }));
   });
 
+  it('exposes the Accept-Patch 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('Accept-Patch');
+  });
+
   it('exposes the Location header via CORS.', async(): Promise<void> => {
     const res = await request(server).get('/').expect(200);
     const exposed = res.header['access-control-expose-headers'];