Changeset 33051 for main/trunk/greenstone3/src/java/org/greenstone/gsdl3/service/GS2Construct.java

Timestamp:

2019-05-06T16:23:43+12:00 (5 years ago)

Author:

ak19

Message:

Tweak to Friday's commit to URL encode param values: can't use nio.StandardCharsets because our release kit JDK is too old to recognise this import, so hardcoding UTF-8 charset name which is what people used to do in the past. Also updated comments. And removed some unnecessary commented out code from GS2PerlConstructor.java

File:

• main/trunk/greenstone3/src/java/org/greenstone/gsdl3/service/GS2Construct.java (modified) (3 diffs)

main/trunk/greenstone3/src/java/org/greenstone/gsdl3/service/GS2Construct.java

@@ -25 +25 @@
 import java.io.UnsupportedEncodingException;
 import java.net.URLEncoder;
-import java.nio.charset.StandardCharsets;
 import java.util.Collections;
 import java.util.Iterator;
@@ -845 +844 @@
                 String paramvalue = (String) entry.getValue();
 
-                // And need to ensure that special characters won't get clobbered on Windows by perl/CGI.pm (https://www.nntp.perl.org/group/perl.perl5.porters/2016/10/msg240120.html), 
-                // URL encode the query_string, as at https://stackoverflow.com/questions/10786042/java-url-encoding-of-query-string-parameters
-        
-                // perl/CGI.pm doesn't like us URL encoding the entire query string such as the equal sign between each paramName and paramValue.
+                // This will be used to set QUERY_STRING in the environment, see build/GS2PerlConstructor.java
+                // Need to also ensure that special characters in param values won't get clobbered on Windows by perl/CGI.pm,
+                // see also https://www.nntp.perl.org/group/perl.perl5.porters/2016/10/msg240120.html
+                // Therefore URL encode CGI param values in the query_string, as at https://stackoverflow.com/questions/10786042/java-url-encoding-of-query-string-parameters
+        
+                // perl/CGI.pm doesn't like us URL encoding the entire QUERY_STRING such as the equal sign between each paramName and paramValue.
                 // So we URL encode each paramValue separately, which is done in GS2Construct.java::runCommand()
                 querystring.append(paramname + "=" + urlEncodeValue(paramname, paramvalue));
@@ -897 +898 @@
         String oldParamVal = paramVal;
         try{
-            paramVal = URLEncoder.encode(paramVal, StandardCharsets.UTF_8.name());          
+            paramVal = URLEncoder.encode(paramVal, "UTF-8");            
+            // Better way as per https://stackoverflow.com/questions/10786042/java-url-encoding-of-query-string-parameters
+            // but our release kits' Java is too old for importing java.nio.charset.StandardCharsets
+            //paramVal = URLEncoder.encode(paramVal, StandardCharsets.UTF_8.name());            
         } catch(UnsupportedEncodingException uee) {
             logger.warn("**** Unable to encode query_string param " + paramName + " in UTF-8, so attempting to continue with its unencoded value."); // don't output param value to log, in case of sensitive data?