From 09ba27543b8726fb61cd998bf9614018a9ba7316 Mon Sep 17 00:00:00 2001
From: Dannon Baker <dannon.baker@gmail.com>
Date: Tue, 25 Apr 2017 19:03:14 -0400
Subject: [PATCH] Slightly adjust logic for discarding headers; this shouldn't
 try to throw away empty headers anymore.

---
 lib/galaxy/web/framework/middleware/remoteuser.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/galaxy/web/framework/middleware/remoteuser.py b/lib/galaxy/web/framework/middleware/remoteuser.py
index 4867a0be667..f359324452e 100644
--- a/lib/galaxy/web/framework/middleware/remoteuser.py
+++ b/lib/galaxy/web/framework/middleware/remoteuser.py
@@ -69,7 +69,7 @@ class RemoteUser( object ):
             assert self.remote_user_header not in environ
             environ[ self.remote_user_header ] = self.single_user
 
-        if environ.get(self.remote_user_header, '(null)').startswith('(null)'):
+        if environ.get(self.remote_user_header, '').startswith('(null)'):
             # Throw away garbage headers.
             # Apache sets REMOTE_USER to the string '(null)' when using the
             # Rewrite* method for passing REMOTE_USER and a user is not authenticated.