Add lock on deleteResource.

RubenVerborgh · RubenVerborgh · commit e9c8db3f18ea · 2019-03-03T15:22:05.000+01:00
diff --git a/lib/ldp.js b/lib/ldp.js
@@ -452,11 +452,17 @@ class LDP {
   }
 
   async deleteResource (path) {
+    let releaseLock
     try {
-      return promisify(fs.unlink)(path)
+      releaseLock = await lock(path, { retries: 10 })
+      return await promisify(fs.unlink)(path)
     } catch (err) {
       debug.container('DELETE -- unlink() error: ' + err)
       throw error(err, 'Failed to delete resource')
+    } finally {
+      if (releaseLock) {
+        await releaseLock()
+      }
     }
   }
 

Original file line number	Diff line number	Diff line change
`@@ -452,11 +452,17 @@ class LDP {`
`452`	`452`	`}`
`453`	`453`
`454`	`454`	`async deleteResource (path) {`
	`455`	`+ let releaseLock`
`455`	`456`	`try {`
`456`		`- return promisify(fs.unlink)(path)`
	`457`	`+ releaseLock = await lock(path, { retries: 10 })`
	`458`	`+ return await promisify(fs.unlink)(path)`
`457`	`459`	`} catch (err) {`
`458`	`460`	`debug.container('DELETE -- unlink() error: ' + err)`
`459`	`461`	`throw error(err, 'Failed to delete resource')`
	`462`	`+ } finally {`
	`463`	`+ if (releaseLock) {`
	`464`	`+ await releaseLock()`
	`465`	`+ }`
`460`	`466`	`}`
`461`	`467`	`}`
`462`	`468`