From 9e95ee5825589c01221144f3153ddaf0f7a0134b Mon Sep 17 00:00:00 2001 From: Misagh Moayyed Date: Wed, 7 Sep 2016 16:10:16 +0430 Subject: [PATCH] Handle date equality when checking for saml assertion validity --- .../java/org/jasig/cas/client/validation/AssertionImpl.java | 3 ++- .../org/jasig/cas/client/validation/AssertionImplTests.java | 3 ++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/cas-client-core/src/main/java/org/jasig/cas/client/validation/AssertionImpl.java b/cas-client-core/src/main/java/org/jasig/cas/client/validation/AssertionImpl.java index 14064c1..da40aba 100644 --- a/cas-client-core/src/main/java/org/jasig/cas/client/validation/AssertionImpl.java +++ b/cas-client-core/src/main/java/org/jasig/cas/client/validation/AssertionImpl.java @@ -133,6 +133,7 @@ public final class AssertionImpl implements Assertion { } final Date now = new Date(); - return this.validFromDate.before(now) && (this.validUntilDate == null || this.validUntilDate.after(now)); + return (this.validFromDate.before(now) || this.validFromDate.equals(now)) + && (this.validUntilDate == null || this.validUntilDate.after(now) || this.validUntilDate.equals(now)); } } diff --git a/cas-client-core/src/test/java/org/jasig/cas/client/validation/AssertionImplTests.java b/cas-client-core/src/test/java/org/jasig/cas/client/validation/AssertionImplTests.java index b175523..335dbe6 100644 --- a/cas-client-core/src/test/java/org/jasig/cas/client/validation/AssertionImplTests.java +++ b/cas-client-core/src/test/java/org/jasig/cas/client/validation/AssertionImplTests.java @@ -18,6 +18,7 @@ */ package org.jasig.cas.client.validation; +import java.util.Date; import java.util.HashMap; import java.util.Map; import junit.framework.TestCase; @@ -50,7 +51,7 @@ public final class AssertionImplTests extends TestCase { } public void testAssertionValidity() throws Exception { - final Assertion assertion = new AssertionImpl(CONST_PRINCIPAL, CONST_ATTRIBUTES); + final Assertion assertion = new AssertionImpl(CONST_PRINCIPAL, new Date(), new Date(), new Date(), CONST_ATTRIBUTES); assertTrue(assertion.isValid()); }