Github user ceharris commented on a diff in the pull request:
https://github.com/apache/guacamole-client/pull/181#discussion_r182706319
--- Diff:
extensions/guacamole-auth-quickconnect/src/main/java/org/apache/guacamole/auth/quickconnect/utility/QCParser.java
---
@@ -131,7 +141,14 @@ public static GuacamoleConfiguration
getConfiguration(String uri)
if (paramList != null) {
for (String parameter : paramList) {
String[] paramArray = parameter.split("=", 2);
- qcConfig.setParameter(paramArray[0],paramArray[1]);
+ try {
+ qcConfig.setParameter(URLDecoder.decode(paramArray[0],
"UTF-8"),
--- End diff --
Might make sense to apply `URLDecoder.decode` to the entire query string
once.
---
