diff --git a/spring-security-kerberos-core/src/main/java/org/springframework/security/kerberos/authentication/KerberosTicketValidation.java b/spring-security-kerberos-core/src/main/java/org/springframework/security/kerberos/authentication/KerberosTicketValidation.java index d2a49ae..bd344e8 100644 --- a/spring-security-kerberos-core/src/main/java/org/springframework/security/kerberos/authentication/KerberosTicketValidation.java +++ b/spring-security-kerberos-core/src/main/java/org/springframework/security/kerberos/authentication/KerberosTicketValidation.java @@ -35,7 +35,7 @@ public class KerberosTicketValidation { } public KerberosTicketValidation(String username, Subject subject, byte[] responseToken, GSSContext gssContext) { - this(username, subject, responseToken, gssContext, null) + this(username, subject, responseToken, gssContext, null); } public KerberosTicketValidation(String username, Subject subject, byte[] responseToken, GSSContext gssContext, GSSCredential delegationCredential) { diff --git a/spring-security-kerberos-core/src/main/java/org/springframework/security/kerberos/authentication/sun/SunJaasKerberosTicketValidator.java b/spring-security-kerberos-core/src/main/java/org/springframework/security/kerberos/authentication/sun/SunJaasKerberosTicketValidator.java index 0bc7630..c576103 100644 --- a/spring-security-kerberos-core/src/main/java/org/springframework/security/kerberos/authentication/sun/SunJaasKerberosTicketValidator.java +++ b/spring-security-kerberos-core/src/main/java/org/springframework/security/kerberos/authentication/sun/SunJaasKerberosTicketValidator.java @@ -295,7 +295,7 @@ public class SunJaasKerberosTicketValidator implements KerberosTicketValidator, this.realmName = realmName; this.multiTier = multiTier; this.debug = debug; - this.refreshKrb5Config = refreshKrb5Configx + this.refreshKrb5Config = refreshKrb5Config; } @Override diff --git a/spring-security-kerberos-core/src/test/java/org/springframework/security/kerberos/authentication/KerberosTicketValidationTest.java b/spring-security-kerberos-core/src/test/java/org/springframework/security/kerberos/authentication/KerberosTicketValidationTest.java new file mode 100644 index 0000000..32ec415 --- /dev/null +++ b/spring-security-kerberos-core/src/test/java/org/springframework/security/kerberos/authentication/KerberosTicketValidationTest.java @@ -0,0 +1,52 @@ +package org.springframework.security.kerberos.authentication; + +import org.ietf.jgss.GSSContext; +import org.ietf.jgss.GSSCredential; +import org.junit.Test; + +import javax.security.auth.Subject; + +import static org.junit.Assert.*; +import static org.mockito.Mockito.*; + +public class KerberosTicketValidationTest { + + private String username = "username"; + private Subject subject = new Subject(); + private byte[] responseToken = "token".getBytes(); + private GSSContext gssContext = mock(GSSContext.class); + private GSSCredential delegationCredential = mock(GSSCredential.class); + + @Test + public void createResultOfTicketValidationWithSubject() { + + KerberosTicketValidation ticketValidation = new KerberosTicketValidation( + username, + subject, + responseToken, + gssContext); + + assertEquals(username, ticketValidation.username()); + assertEquals(responseToken, ticketValidation.responseToken()); + assertEquals(gssContext, ticketValidation.getGssContext()); + + assertNull("With no credential delegation", ticketValidation.getDelegationCredential()); + } + + @Test + public void createResultOfTicketValidationWithSubjectAndDelegation() { + + KerberosTicketValidation ticketValidation = new KerberosTicketValidation( + username, + subject, + responseToken, + gssContext, + delegationCredential); + + assertEquals(username, ticketValidation.username()); + assertEquals(responseToken, ticketValidation.responseToken()); + assertEquals(gssContext, ticketValidation.getGssContext()); + + assertEquals("With no credential delegation", delegationCredential, ticketValidation.getDelegationCredential()); + } +} \ No newline at end of file