Author: markt
Date: Wed Jan 13 14:17:26 2016
New Revision: 1724430

URL: http://svn.apache.org/viewvc?rev=1724430&view=rev
Log:
Remove deprecated code.
Clean-up

Modified:
    tomcat/trunk/java/org/apache/catalina/util/LocalStrings.properties
    tomcat/trunk/java/org/apache/catalina/util/RequestUtil.java

Modified: tomcat/trunk/java/org/apache/catalina/util/LocalStrings.properties
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/util/LocalStrings.properties?rev=1724430&r1=1724429&r2=1724430&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/util/LocalStrings.properties 
(original)
+++ tomcat/trunk/java/org/apache/catalina/util/LocalStrings.properties Wed Jan 
13 14:17:26 2016
@@ -36,8 +36,6 @@ lifecycleBase.stopFail=Failed to stop co
 lifecycleMBeanBase.registerFail=Failed to register object [{0}] with name 
[{0}] during component initialisation
 lifecycleMBeanBase.unregisterFail=Failed to unregister MBean with name [{0}] 
during component destruction
 lifecycleMBeanBase.unregisterNoServer=No MBean server was available to 
unregister the MBean [{0}]
-requestUtil.convertHexDigit.notHex=[{0}] is not a hexadecimal digit
-requestUtil.parseParameters.uee=Unable to parse the parameters since the 
encoding [{0}] is not supported.
 SecurityUtil.doAsPrivilege=An exception occurs when running the 
PrivilegedExceptionAction block.
 sessionIdGeneratorBase.createRandom=Creation of SecureRandom instance for 
session ID generation using [{0}] took [{1}] milliseconds.
 sessionIdGeneratorBase.random=Exception initializing random number generator 
of class [{0}]. Falling back to java.secure.SecureRandom

Modified: tomcat/trunk/java/org/apache/catalina/util/RequestUtil.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/util/RequestUtil.java?rev=1724430&r1=1724429&r2=1724430&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/util/RequestUtil.java (original)
+++ tomcat/trunk/java/org/apache/catalina/util/RequestUtil.java Wed Jan 13 
14:17:26 2016
@@ -16,16 +16,6 @@
  */
 package org.apache.catalina.util;
 
-import java.io.UnsupportedEncodingException;
-import java.nio.charset.Charset;
-import java.util.Map;
-
-import org.apache.juli.logging.Log;
-import org.apache.juli.logging.LogFactory;
-import org.apache.tomcat.util.buf.B2CConverter;
-import org.apache.tomcat.util.res.StringManager;
-
-
 /**
  * General purpose request parsing and encoding utility methods.
  *
@@ -34,16 +24,6 @@ import org.apache.tomcat.util.res.String
  */
 public final class RequestUtil {
 
-
-    private static final Log log = LogFactory.getLog(RequestUtil.class);
-
-    /**
-     * The string resources for this package.
-     */
-    private static final StringManager sm =
-        StringManager.getManager("org.apache.catalina.util");
-
-
     /**
      * Filter the specified message string for characters that are sensitive
      * in HTML.  This avoids potential attacks caused by including JavaScript
@@ -55,8 +35,9 @@ public final class RequestUtil {
      */
     public static String filter(String message) {
 
-        if (message == null)
-            return (null);
+        if (message == null) {
+            return null;
+        }
 
         char content[] = new char[message.length()];
         message.getChars(0, message.length(), content, 0);
@@ -79,163 +60,6 @@ public final class RequestUtil {
                 result.append(content[i]);
             }
         }
-        return (result.toString());
-
-    }
-
-
-    /**
-     * Append request parameters from the specified String to the specified
-     * Map.  It is presumed that the specified Map is not accessed from any
-     * other thread, so no synchronization is performed.
-     * <p>
-     * <strong>IMPLEMENTATION NOTE</strong>:  URL decoding is performed
-     * individually on the parsed name and value elements, rather than on
-     * the entire query string ahead of time, to properly deal with the case
-     * where the name or value includes an encoded "=" or "&amp;" character
-     * that would otherwise be interpreted as a delimiter.
-     *
-     * @param map Map that accumulates the resulting parameters
-     * @param data Input string containing request parameters
-     * @param encoding The encoding to use; encoding must not be null.
-     * If an unsupported encoding is specified the parameters will not be
-     * parsed and the map will not be modified
-     *
-     * @deprecated Unused. This will be removed in Tomcat 9.0.x.
-     */
-    @Deprecated
-    public static void parseParameters(Map<String,String[]> map, String data,
-            String encoding) {
-
-        if ((data != null) && (data.length() > 0)) {
-
-            // use the specified encoding to extract bytes out of the
-            // given string so that the encoding is not lost.
-            byte[] bytes = null;
-            try {
-                bytes = data.getBytes(B2CConverter.getCharset(encoding));
-                parseParameters(map, bytes, encoding);
-            } catch (UnsupportedEncodingException uee) {
-                if (log.isDebugEnabled()) {
-                    log.debug(sm.getString("requestUtil.parseParameters.uee",
-                            encoding), uee);
-                }
-            }
-
-        }
-
-    }
-
-
-    /**
-     * Convert a byte character value to hexadecimal digit value.
-     *
-     * @param b the character value byte
-     */
-    private static byte convertHexDigit( byte b ) {
-        if ((b >= '0') && (b <= '9')) return (byte)(b - '0');
-        if ((b >= 'a') && (b <= 'f')) return (byte)(b - 'a' + 10);
-        if ((b >= 'A') && (b <= 'F')) return (byte)(b - 'A' + 10);
-        throw new IllegalArgumentException(
-                sm.getString("requestUtil.convertHexDigit.notHex",
-                        Character.valueOf((char)b)));
-    }
-
-
-    /**
-     * Put name and value pair in map.  When name already exist, add value
-     * to array of values.
-     *
-     * @param map The map to populate
-     * @param name The parameter name
-     * @param value The parameter value
-     */
-    private static void putMapEntry( Map<String,String[]> map, String name,
-            String value) {
-        String[] newValues = null;
-        String[] oldValues = map.get(name);
-        if (oldValues == null) {
-            newValues = new String[1];
-            newValues[0] = value;
-        } else {
-            newValues = new String[oldValues.length + 1];
-            System.arraycopy(oldValues, 0, newValues, 0, oldValues.length);
-            newValues[oldValues.length] = value;
-        }
-        map.put(name, newValues);
-    }
-
-
-    /**
-     * Append request parameters from the specified String to the specified
-     * Map.  It is presumed that the specified Map is not accessed from any
-     * other thread, so no synchronization is performed.
-     * <p>
-     * <strong>IMPLEMENTATION NOTE</strong>:  URL decoding is performed
-     * individually on the parsed name and value elements, rather than on
-     * the entire query string ahead of time, to properly deal with the case
-     * where the name or value includes an encoded "=" or "&amp;" character
-     * that would otherwise be interpreted as a delimiter.
-     *
-     * NOTE: byte array data is modified by this method.  Caller beware.
-     *
-     * @param map Map that accumulates the resulting parameters
-     * @param data Input string containing request parameters
-     * @param encoding The encoding to use; if null, the default encoding is
-     * used
-     *
-     * @exception UnsupportedEncodingException if the requested encoding is not
-     * supported.
-     *
-     * @deprecated Unused. This will be removed in Tomcat 9.0.x.
-     */
-    @Deprecated
-    public static void parseParameters(Map<String,String[]> map, byte[] data,
-            String encoding) throws UnsupportedEncodingException {
-
-        Charset charset = B2CConverter.getCharset(encoding);
-
-        if (data != null && data.length > 0) {
-            int    ix = 0;
-            int    ox = 0;
-            String key = null;
-            String value = null;
-            while (ix < data.length) {
-                byte c = data[ix++];
-                switch ((char) c) {
-                case '&':
-                    value = new String(data, 0, ox, charset);
-                    if (key != null) {
-                        putMapEntry(map, key, value);
-                        key = null;
-                    }
-                    ox = 0;
-                    break;
-                case '=':
-                    if (key == null) {
-                        key = new String(data, 0, ox, charset);
-                        ox = 0;
-                    } else {
-                        data[ox++] = c;
-                    }
-                    break;
-                case '+':
-                    data[ox++] = (byte)' ';
-                    break;
-                case '%':
-                    data[ox++] = (byte)((convertHexDigit(data[ix++]) << 4)
-                                    + convertHexDigit(data[ix++]));
-                    break;
-                default:
-                    data[ox++] = c;
-                }
-            }
-            //The last value does not end in '&'.  So save it now.
-            if (key != null) {
-                value = new String(data, 0, ox, charset);
-                putMapEntry(map, key, value);
-            }
-        }
-
+        return result.toString();
     }
 }



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to