URLDecode remaining /vault/ and /open/ paths. Fixes #138.

coddingtonbear · coddingtonbear · commit 8700c116a6bf · 2024-11-19T10:31:18.000-06:00
diff --git a/src/requestHandler.ts b/src/requestHandler.ts
@@ -332,7 +332,9 @@ export default class RequestHandler {
   }
 
   async vaultGet(req: express.Request, res: express.Response): Promise<void> {
-    const path = req.path.slice(req.path.indexOf("/", 1) + 1);
+    const path = decodeURIComponent(
+      req.path.slice(req.path.indexOf("/", 1) + 1)
+    );
 
     return this._vaultGet(path, req, res);
   }
@@ -1060,7 +1062,9 @@ export default class RequestHandler {
   }
 
   async openPost(req: express.Request, res: express.Response): Promise<void> {
-    const path = req.path.slice(req.path.indexOf("/", 1) + 1);
+    const path = decodeURIComponent(
+      req.path.slice(req.path.indexOf("/", 1) + 1)
+    );
 
     const query = queryString.parseUrl(req.originalUrl, {
       parseBooleans: true,

Original file line number	Diff line number	Diff line change
`@@ -332,7 +332,9 @@ export default class RequestHandler {`
`332`	`332`	`}`
`333`	`333`
`334`	`334`	`async vaultGet(req: express.Request, res: express.Response): Promise<void> {`
`335`		`- const path = req.path.slice(req.path.indexOf("/", 1) + 1);`
	`335`	`+ const path = decodeURIComponent(`
	`336`	`+ req.path.slice(req.path.indexOf("/", 1) + 1)`
	`337`	`+ );`
`336`	`338`
`337`	`339`	`return this._vaultGet(path, req, res);`
`338`	`340`	`}`
`@@ -1060,7 +1062,9 @@ export default class RequestHandler {`
`1060`	`1062`	`}`
`1061`	`1063`
`1062`	`1064`	`async openPost(req: express.Request, res: express.Response): Promise<void> {`
`1063`		`- const path = req.path.slice(req.path.indexOf("/", 1) + 1);`
	`1065`	`+ const path = decodeURIComponent(`
	`1066`	`+ req.path.slice(req.path.indexOf("/", 1) + 1)`
	`1067`	`+ );`
`1064`	`1068`
`1065`	`1069`	`const query = queryString.parseUrl(req.originalUrl, {`
`1066`	`1070`	`parseBooleans: true,`