Changeset 33051 for main/trunk/greenstone3/src/java/org/greenstone/gsdl3/build/GS2PerlConstructor.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/build/GS2PerlConstructor.java (modified) (2 diffs)

main/trunk/greenstone3/src/java/org/greenstone/gsdl3/build/GS2PerlConstructor.java

@@ -20 +20 @@
 import java.io.InputStream;
 import java.io.IOException;
-//import java.io.UnsupportedEncodingException;
-//import java.net.URLEncoder;
-//import java.nio.charset.StandardCharsets;
 import java.util.ArrayList;
 import java.util.Vector;
@@ -334 +331 @@
         // http://www.cgi101.com/class/ch3/text.html
         
-        // 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.
-        // So we URL encode each paramValue separately, which is done in GS2Construct.java::runCommand()
-        /*
-        String old_query_string = this.query_string;
-        try{
-            this.query_string = URLEncoder.encode(this.query_string, StandardCharsets.UTF_8.name());
-            //this.query_string = this.query_string.replace("+","%2B"); // https://stackoverflow.com/questions/1211229/in-a-url-should-spaces-be-encoded-using-20-or
-        } catch(UnsupportedEncodingException uee) {
-            logger.warn("**** Unable to encode query_string in UTF-8, so attempting to continue with the unencoded value of query_string");
-            this.query_string = old_query_string; 
-        }
-        */
+        // And since we need to ensure that special characters won't get clobbered on Windows by perl/CGI.pm
+        // (see https://www.nntp.perl.org/group/perl.perl5.porters/2016/10/msg240120.html), 
+        // each param *value* in QUERY_STRING, not QUERY_STRING in entirety, is further URL encoded,
+        // as at https://stackoverflow.com/questions/10786042/java-url-encoding-of-query-string-parameters
+        // This URL encoding of paramValues already happened in GS2Construct.java::runCommand(), so QUERY_STRING is ready now.
         
         String[] envvars = {