fixed double encoding of bracket url param

dppeak · quickthyme · commit b82eb4416d60 · 2019-11-04T13:20:34.000-06:00
Signed-off-by: Quickthyme &lt;quickthyme@mac.com&gt;
diff --git a/Sources/WebRequest/WebRequest+URLEncoder.swift b/Sources/WebRequest/WebRequest+URLEncoder.swift
@@ -21,7 +21,7 @@ extension WebRequest : URLEncoder {
     }
 
     static func urlEncode(string: String) -> String {
-        let allowed1 = CharacterSet(charactersIn: "-._~/?@")
+        let allowed1 = CharacterSet(charactersIn: "-._~/?@[]")
         let allowed2 = CharacterSet.alphanumerics
         return string.addingPercentEncoding(withAllowedCharacters: allowed1.union(allowed2)) ?? ""
     }
@@ -35,7 +35,7 @@ extension WebRequest : URLEncoder {
     }
 
     static func urlFormEncode(string: String) -> String {
-        let allowed1 = CharacterSet(charactersIn: "*-._@ ")
+        let allowed1 = CharacterSet(charactersIn: "*-._@[] ")
         let allowed2 = CharacterSet.alphanumerics
         return (string.addingPercentEncoding(withAllowedCharacters: allowed1.union(allowed2)) ?? "")
             .replacingOccurrences(of: " ", with: "+")

Original file line number	Diff line number	Diff line change
`@@ -21,7 +21,7 @@ extension WebRequest : URLEncoder {`
`21`	`21`	`}`
`22`	`22`
`23`	`23`	`static func urlEncode(string: String) -> String {`
`24`		`- let allowed1 = CharacterSet(charactersIn: "-._~/?@")`
	`24`	`+ let allowed1 = CharacterSet(charactersIn: "-._~/?@[]")`
`25`	`25`	`let allowed2 = CharacterSet.alphanumerics`
`26`	`26`	`return string.addingPercentEncoding(withAllowedCharacters: allowed1.union(allowed2)) ?? ""`
`27`	`27`	`}`
`@@ -35,7 +35,7 @@ extension WebRequest : URLEncoder {`
`35`	`35`	`}`
`36`	`36`
`37`	`37`	`static func urlFormEncode(string: String) -> String {`
`38`		`- let allowed1 = CharacterSet(charactersIn: "*-._@ ")`
	`38`	`+ let allowed1 = CharacterSet(charactersIn: "*-._@[] ")`
`39`	`39`	`let allowed2 = CharacterSet.alphanumerics`
`40`	`40`	`return (string.addingPercentEncoding(withAllowedCharacters: allowed1.union(allowed2)) ?? "")`
`41`	`41`	`.replacingOccurrences(of: " ", with: "+")`