Return-Path: X-Original-To: apmail-hadoop-common-issues-archive@minotaur.apache.org Delivered-To: apmail-hadoop-common-issues-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 152808551 for ; Wed, 10 Aug 2011 13:55:53 +0000 (UTC) Received: (qmail 65308 invoked by uid 500); 10 Aug 2011 13:55:52 -0000 Delivered-To: apmail-hadoop-common-issues-archive@hadoop.apache.org Received: (qmail 64911 invoked by uid 500); 10 Aug 2011 13:55:51 -0000 Mailing-List: contact common-issues-help@hadoop.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: common-issues@hadoop.apache.org Delivered-To: mailing list common-issues@hadoop.apache.org Received: (qmail 64878 invoked by uid 99); 10 Aug 2011 13:55:51 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 10 Aug 2011 13:55:51 +0000 X-ASF-Spam-Status: No, hits=-2000.8 required=5.0 tests=ALL_TRUSTED,RP_MATCHES_RCVD X-Spam-Check-By: apache.org Received: from [140.211.11.116] (HELO hel.zones.apache.org) (140.211.11.116) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 10 Aug 2011 13:55:48 +0000 Received: from hel.zones.apache.org (hel.zones.apache.org [140.211.11.116]) by hel.zones.apache.org (Postfix) with ESMTP id A6BC5B5712 for ; Wed, 10 Aug 2011 13:55:27 +0000 (UTC) Date: Wed, 10 Aug 2011 13:55:27 +0000 (UTC) From: "Daryn Sharp (JIRA)" To: common-issues@hadoop.apache.org Message-ID: <66150925.23761.1312984527679.JavaMail.tomcat@hel.zones.apache.org> In-Reply-To: <1703861590.21338.1312920387446.JavaMail.tomcat@hel.zones.apache.org> Subject: [jira] [Commented] (HADOOP-7531) Add servlet util methods for handling paths in requests MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 X-Virus-Checked: Checked by ClamAV on apache.org [ https://issues.apache.org/jira/browse/HADOOP-7531?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13082346#comment-13082346 ] Daryn Sharp commented on HADOOP-7531: ------------------------------------- bq. And we wouldn't let someone check that in.. =) I'm worried because it's a latent land-mine. Over 20 years I've debugged so many "it'll never happen" bugs that I strongly adhere to Murphy's Law. Someday the charset will be changed, either in error or with legit purpose, or an external jar will fiddle with the charset, or a deployment env will use an incompatible charset, etc. When that day arrives, some poor developer will spend too much time debugging why the web page acts goofy instead of immediately seeing an invalid charset exception. Perhaps the safest way is to not catch the exception and use {{URIUtil.encodeXXX(String, "UTF-8")}}? > Add servlet util methods for handling paths in requests > -------------------------------------------------------- > > Key: HADOOP-7531 > URL: https://issues.apache.org/jira/browse/HADOOP-7531 > Project: Hadoop Common > Issue Type: Improvement > Components: util > Affects Versions: 0.23.0 > Reporter: Eli Collins > Assignee: Eli Collins > Fix For: 0.23.0 > > Attachments: hadoop-7531-1.patch, hadoop-7531-2.patch > > > Common side of HDFS-2235. -- This message is automatically generated by JIRA. For more information on JIRA, see: http://www.atlassian.com/software/jira