Repository: hadoop
Updated Branches:
  refs/heads/branch-2 8de604584 -> db4419df6


HADOOP-11859. PseudoAuthenticationHandler fails with httpcomponents v4.4. 
Contributed by Eugene Koifman.


Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/db4419df
Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/db4419df
Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/db4419df

Branch: refs/heads/branch-2
Commit: db4419df6bc72e9a1ddd90bac4dd7dcf0fb05479
Parents: 8de6045
Author: Jitendra Pandey <jiten...@apache.org>
Authored: Wed Apr 22 09:59:12 2015 -0700
Committer: Jitendra Pandey <jiten...@apache.org>
Committed: Wed Apr 22 10:02:02 2015 -0700

----------------------------------------------------------------------
 .../authentication/server/PseudoAuthenticationHandler.java     | 6 +++++-
 hadoop-common-project/hadoop-common/CHANGES.txt                | 3 +++
 2 files changed, 8 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hadoop/blob/db4419df/hadoop-common-project/hadoop-auth/src/main/java/org/apache/hadoop/security/authentication/server/PseudoAuthenticationHandler.java
----------------------------------------------------------------------
diff --git 
a/hadoop-common-project/hadoop-auth/src/main/java/org/apache/hadoop/security/authentication/server/PseudoAuthenticationHandler.java
 
b/hadoop-common-project/hadoop-auth/src/main/java/org/apache/hadoop/security/authentication/server/PseudoAuthenticationHandler.java
index 50f0cf1..19947aa 100644
--- 
a/hadoop-common-project/hadoop-auth/src/main/java/org/apache/hadoop/security/authentication/server/PseudoAuthenticationHandler.java
+++ 
b/hadoop-common-project/hadoop-auth/src/main/java/org/apache/hadoop/security/authentication/server/PseudoAuthenticationHandler.java
@@ -142,7 +142,11 @@ public class PseudoAuthenticationHandler implements 
AuthenticationHandler {
   }
 
   private String getUserName(HttpServletRequest request) {
-    List<NameValuePair> list = URLEncodedUtils.parse(request.getQueryString(), 
UTF8_CHARSET);
+    String queryString = request.getQueryString();
+    if(queryString == null || queryString.length() == 0) {
+      return null;
+    }
+    List<NameValuePair> list = URLEncodedUtils.parse(queryString, 
UTF8_CHARSET);
     if (list != null) {
       for (NameValuePair nv : list) {
         if (PseudoAuthenticator.USER_NAME.equals(nv.getName())) {

http://git-wip-us.apache.org/repos/asf/hadoop/blob/db4419df/hadoop-common-project/hadoop-common/CHANGES.txt
----------------------------------------------------------------------
diff --git a/hadoop-common-project/hadoop-common/CHANGES.txt 
b/hadoop-common-project/hadoop-common/CHANGES.txt
index f6580f5..8bcc3eb 100644
--- a/hadoop-common-project/hadoop-common/CHANGES.txt
+++ b/hadoop-common-project/hadoop-common/CHANGES.txt
@@ -92,6 +92,9 @@ Release 2.8.0 - UNRELEASED
     HADOOP-11846 TestCertificateUtil.testCorruptPEM failing on Jenkins JDK8.
     (Larry McCay via stevel)
 
+    HADOOP-11859. PseudoAuthenticationHandler fails with httpcomponents v4.4.
+    (Eugene Koifman via jitendra)
+
 Release 2.7.1 - UNRELEASED
 
   INCOMPATIBLE CHANGES

Reply via email to