Accept-Patch Accept-Post Accept-Put headers

lib/handlers/get.js

@@ -27,8 +27,12 @@ async function handler (req, res, next) {
   const requestedType = negotiator.mediaType()
   const possibleRDFType = negotiator.mediaType(RDFs)
 
+  // deprecated kept for compatibility
   res.header('MS-Author-Via', 'SPARQL')
 
+  res.header('Accept-Patch', 'text/n3, application/sparql-update, application/sparql-update-single-match')
+  res.header('Accept-Post', '*/*')
+
   // Set live updates
   if (ldp.live) {
     res.header('Updates-Via', ldp.resourceMapper.resolveUrl(req.hostname).replace(/^http/, 'ws'))

lib/handlers/patch.js

@@ -39,7 +39,6 @@ function contentForNew (contentType) {
 // Handles a PATCH request
 async function patchHandler (req, res, next) {
   debug(`PATCH -- ${req.originalUrl}`)
-  res.header('MS-Author-Via', 'SPARQL')
   try {
     // Obtain details of the target resource
     const ldp = req.app.locals.ldp
@@ -70,7 +69,7 @@ async function patchHandler (req, res, next) {
     if (!parsePatch) {
       throw error(415, `Unsupported patch content type: ${patch.contentType}`)
     }
-
+    res.header('Accept-Patch', patch.contentType) // is this needed ?
     // Parse the patch document and verify permissions
     const patchObject = await parsePatch(url, patch.uri, patch.text)
     await checkPermission(req, patchObject, resourceExists)

test/integration/header-test.js

@@ -20,7 +20,7 @@ describe('Header handler', () => {
     request = supertest(server)
   })
 
-  describe('MS-Author-Via', () => {
+  describe('MS-Author-Via', () => { // deprecated
     describeHeaderTest('read/append for the public', {
       resource: '/public-ra',
       headers: {
@@ -30,6 +30,26 @@ describe('Header handler', () => {
     })
   })
 
+  describe('Accept-Patch', () => {
+    describeHeaderTest('read/append for the public', {
+      resource: '/public-ra',
+      headers: {
+        'Accept-Patch': 'text/n3, application/sparql-update, application/sparql-update-single-match',
+        'Access-Control-Expose-Headers': /(^|,\s*)Accept-Patch(,|$)/
+      }
+    })
+  })
+
+  describe('Accept-Post', () => {
+    describeHeaderTest('read/append for the public', {
+      resource: '/public-ra',
+      headers: {
+        'Accept-Post': '*/*',
+        'Access-Control-Expose-Headers': /(^|,\s*)Accept-Post(,|$)/
+      }
+    })
+  })
+
   describe('WAC-Allow', () => {
     describeHeaderTest('read/append for the public', {
       resource: '/public-ra',

test/integration/patch-test.js

@@ -22,7 +22,7 @@ const serverOptions = {
   forceUser: `${serverUri}/profile/card#me`
 }
 
-describe('PATCH', () => {
+describe('PATCH through text/n3', () => {
   let request
   let server