Fixes #250

Modified test Signed-off-by: Guillermo González de Agüero <[email protected]>
eclipse-ee4j · Jul 10, 2019 · 2c21f6e · 2c21f6e
1 parent 0156152
commit 2c21f6e
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 7 deletions.
diff --git a/impl/src/main/java/org/glassfish/soteria/mechanisms/BasicAuthenticationMechanism.java b/impl/src/main/java/org/glassfish/soteria/mechanisms/BasicAuthenticationMechanism.java
@@ -85,7 +85,7 @@ private String[] getCredentials(HttpServletRequest request) {
 
 		String authorizationHeader = request.getHeader("Authorization");
 		if (!isEmpty(authorizationHeader) && authorizationHeader.startsWith("Basic ") ) {
-			return new String(parseBase64Binary(authorizationHeader.substring(6))).split(":");
+			return new String(parseBase64Binary(authorizationHeader.substring(6))).split(":", 2);
 		}
 
 		return null;

diff --git a/test/app-mem-basic/src/main/java/test/TestIdentityStore.java b/test/app-mem-basic/src/main/java/test/TestIdentityStore.java
@@ -17,21 +17,19 @@
 package test;
 
 import static java.util.Arrays.asList;
-import static javax.security.enterprise.identitystore.CredentialValidationResult.INVALID_RESULT;
-
 import java.util.HashSet;
-
 import javax.enterprise.context.ApplicationScoped;
 import javax.security.enterprise.credential.UsernamePasswordCredential;
 import javax.security.enterprise.identitystore.CredentialValidationResult;
+import static javax.security.enterprise.identitystore.CredentialValidationResult.INVALID_RESULT;
 import javax.security.enterprise.identitystore.IdentityStore;
 
 @ApplicationScoped
 public class TestIdentityStore implements IdentityStore {
 
     public CredentialValidationResult validate(UsernamePasswordCredential usernamePasswordCredential) {
 
-        if (usernamePasswordCredential.compareTo("reza", "secret1")) {
+        if (usernamePasswordCredential.compareTo("reza", "secret:1")) {
             return new CredentialValidationResult("reza", new HashSet<>(asList("foo", "bar")));
         }
 

diff --git a/test/app-mem-basic/src/test/java/org/glassfish/soteria/test/AppMemBasicIT.java b/test/app-mem-basic/src/test/java/org/glassfish/soteria/test/AppMemBasicIT.java
@@ -44,7 +44,7 @@ public static Archive<?> createDeployment() {
     public void testAuthenticated() {
 
     	DefaultCredentialsProvider credentialsProvider = new DefaultCredentialsProvider();
-    	credentialsProvider.addCredentials("reza", "secret1");
+    	credentialsProvider.addCredentials("reza", "secret:1");
 
     	getWebClient().setCredentialsProvider(credentialsProvider);
 
@@ -71,7 +71,7 @@ public void testNotAuthenticated() {
     public void testNotAuthenticatedWrongName() {
 
     	DefaultCredentialsProvider credentialsProvider = new DefaultCredentialsProvider();
-    	credentialsProvider.addCredentials("romo", "secret1");
+    	credentialsProvider.addCredentials("romo", "secret:1");
 
     	getWebClient().setCredentialsProvider(credentialsProvider);