adam/guacamole-client
1
0 Fork 0
Code Issues Pull Requests 1 Packages Releases Wiki Activity

GUACAMOLE-195: Remove call to getRemoteUser() and just use the configured HTTP header.

Browse Source
This commit is contained in:
Nick Couchman committed 2017-02-03 13:13:02 -05:00
1 parent 96fb8a9c6b
commit 8eb33b5ca6
1 file changed
+3 -6
Download Patch File Download Diff File Expand all files Collapse all files
extensions/guacamole-auth-header/src/main/java/org/apache/guacamole/auth/header/AuthenticationProviderService.java
+3 -6
View File
@@ -70,13 +70,10 @@ public class AuthenticationProviderService {
// Pull HTTP header from request if present // Pull HTTP header from request if present
HttpServletRequest request = credentials.getRequest(); HttpServletRequest request = credentials.getRequest();
if(request != null) { if(request != null) {
// Try getRemoteUser(), first // Get the username from the header configured in guacamole.properties
String username = request.getRemoteUser(); String username = request.getHeader(confService.getHttpAuthHeader());
// Check if that worked, if not, try the configured header.
if(username == null)
username = request.getHeader(confService.getHttpAuthHeader());
if (username != null) { if(username != null) {
AuthenticatedUser authenticatedUser = authenticatedUserProvider.get(); AuthenticatedUser authenticatedUser = authenticatedUserProvider.get();
authenticatedUser.init(username, credentials); authenticatedUser.init(username, credentials);
return authenticatedUser; return authenticatedUser;