Merge pull request #218 from gregolsky/v4.1

gregolsky · web-flow · commit 1b111e79ee15 · 2020-04-09T15:48:39.000+02:00
encode key in RemoveCompareExchangeCommand
diff --git a/src/Documents/Operations/CompareExchange/DeleteCompareExchangeValueOperation.ts b/src/Documents/Operations/CompareExchange/DeleteCompareExchangeValueOperation.ts
@@ -57,7 +57,8 @@ export class RemoveCompareExchangeCommand<T> extends RavenCommand<CompareExchang
     }
 
     public createRequest(node: ServerNode): HttpRequestParameters {
-        const uri = node.url + "/databases/" + node.database + "/cmpxchg?key=" + this._key + "&index=" + this._index;
+        const uri = node.url + "/databases/" + node.database + "/cmpxchg?key=" + encodeURIComponent(this._key) 
+            + "&index=" + this._index;
         return {
             method: "DELETE",
             uri

Original file line number	Diff line number	Diff line change
`@@ -57,7 +57,8 @@ export class RemoveCompareExchangeCommand<T> extends RavenCommand<CompareExchang`
`57`	`57`	`}`
`58`	`58`
`59`	`59`	`public createRequest(node: ServerNode): HttpRequestParameters {`
`60`		`- const uri = node.url + "/databases/" + node.database + "/cmpxchg?key=" + this._key + "&index=" + this._index;`
	`60`	`+ const uri = node.url + "/databases/" + node.database + "/cmpxchg?key=" + encodeURIComponent(this._key)`
	`61`	`+ + "&index=" + this._index;`
`61`	`62`	`return {`
`62`	`63`	`method: "DELETE",`
`63`	`64`	`uri`