[java-identity-provider] branch master updated: Replace deprecated AuthnContext methods, add more logging.
Scott Cantor
cantor.2 at osu.edu
Mon Nov 18 12:01:09 EST 2019
This is an automated email from the git hooks/post-receive script.
scantor pushed a commit to branch master
in repository java-identity-provider.
View the commit online:
http://git.shibboleth.net/view/?p=java-identity-provider.git;a=commit;h=f6affce5a7cb68097d816f83ad72e3cb704600b9
The following commit(s) were added to refs/heads/master by this push:
new f6affce Replace deprecated AuthnContext methods, add more logging.
f6affce is described below
commit f6affce5a7cb68097d816f83ad72e3cb704600b9
Author: Scott Cantor <cantor.2 at osu.edu>
AuthorDate: Mon Nov 18 12:01:03 2019 -0500
Replace deprecated AuthnContext methods, add more logging.
---
.../test/flows/saml2/AbstractSAML2SSOFlowTest.java | 2 +-
.../flows/saml2/SAML2TestResponseValidator.java | 2 +-
.../principal/AuthnContextClassRefPrincipal.java | 2 +-
.../principal/AuthnContextDeclRefPrincipal.java | 2 +-
.../AuthnContextClassRefPrincipalTest.java | 2 +-
.../AuthnContextDeclRefPrincipalTest.java | 2 +-
.../audit/impl/AuthnContextAuditExtractor.java | 4 ++--
.../MapDrivenAuthnContextTranslationStrategy.java | 11 ++++-------
.../profile/impl/ProcessRequestedAuthnContext.java | 16 ++++++++--------
.../profile/impl/ValidateSAMLAuthentication.java | 22 ++++++++++++++--------
.../saml2/profile/impl/AddAuthnRequestTest.java | 8 ++++----
.../impl/AddAuthnStatementToAssertionTest.java | 4 ++--
.../impl/ProcessRequestedAuthnContextTest.java | 18 +++++++++---------
13 files changed, 49 insertions(+), 46 deletions(-)
diff --git a/idp-conf/src/test/java/net/shibboleth/idp/test/flows/saml2/AbstractSAML2SSOFlowTest.java b/idp-conf/src/test/java/net/shibboleth/idp/test/flows/saml2/AbstractSAML2SSOFlowTest.java
index 2d64593..0be613a 100644
--- a/idp-conf/src/test/java/net/shibboleth/idp/test/flows/saml2/AbstractSAML2SSOFlowTest.java
+++ b/idp-conf/src/test/java/net/shibboleth/idp/test/flows/saml2/AbstractSAML2SSOFlowTest.java
@@ -141,7 +141,7 @@ public abstract class AbstractSAML2SSOFlowTest extends AbstractSAML2FlowTest {
final AuthnContextClassRef ac =
(AuthnContextClassRef) builderFactory.getBuilder(AuthnContextClassRef.DEFAULT_ELEMENT_NAME).buildObject(
AuthnContextClassRef.DEFAULT_ELEMENT_NAME);
- ac.setAuthnContextClassRef(AuthnContext.UNSPECIFIED_AUTHN_CTX);
+ ac.setURI(AuthnContext.UNSPECIFIED_AUTHN_CTX);
reqAC.getAuthnContextClassRefs().add(ac);
authnRequest.setRequestedAuthnContext(reqAC);
diff --git a/idp-conf/src/test/java/net/shibboleth/idp/test/flows/saml2/SAML2TestResponseValidator.java b/idp-conf/src/test/java/net/shibboleth/idp/test/flows/saml2/SAML2TestResponseValidator.java
index 2fa7751..edc8568 100644
--- a/idp-conf/src/test/java/net/shibboleth/idp/test/flows/saml2/SAML2TestResponseValidator.java
+++ b/idp-conf/src/test/java/net/shibboleth/idp/test/flows/saml2/SAML2TestResponseValidator.java
@@ -537,7 +537,7 @@ public class SAML2TestResponseValidator extends SAML2TestStatusResponseTypeValid
* @param authnContext the authn context
*/
public void assertAuthnContextClassRef(@Nullable final AuthnContextClassRef authnContext) {
- Assert.assertEquals(authnContext.getAuthnContextClassRef(), authnContextClassRef);
+ Assert.assertEquals(authnContext.getURI(), authnContextClassRef);
}
/**
diff --git a/idp-saml-api/src/main/java/net/shibboleth/idp/saml/authn/principal/AuthnContextClassRefPrincipal.java b/idp-saml-api/src/main/java/net/shibboleth/idp/saml/authn/principal/AuthnContextClassRefPrincipal.java
index 0746eec..1b28457 100644
--- a/idp-saml-api/src/main/java/net/shibboleth/idp/saml/authn/principal/AuthnContextClassRefPrincipal.java
+++ b/idp-saml-api/src/main/java/net/shibboleth/idp/saml/authn/principal/AuthnContextClassRefPrincipal.java
@@ -61,7 +61,7 @@ public final class AuthnContextClassRefPrincipal implements CloneablePrincipal {
final AuthnContextClassRef ref = (AuthnContextClassRef) Constraint.isNotNull(
XMLObjectSupport.getBuilder(AuthnContextClassRef.DEFAULT_ELEMENT_NAME),
"No builder for AuthnContextClassRef").buildObject(AuthnContextClassRef.DEFAULT_ELEMENT_NAME);
- ref.setAuthnContextClassRef(getName());
+ ref.setURI(getName());
return ref;
}
diff --git a/idp-saml-api/src/main/java/net/shibboleth/idp/saml/authn/principal/AuthnContextDeclRefPrincipal.java b/idp-saml-api/src/main/java/net/shibboleth/idp/saml/authn/principal/AuthnContextDeclRefPrincipal.java
index aacce05..53bd67c 100644
--- a/idp-saml-api/src/main/java/net/shibboleth/idp/saml/authn/principal/AuthnContextDeclRefPrincipal.java
+++ b/idp-saml-api/src/main/java/net/shibboleth/idp/saml/authn/principal/AuthnContextDeclRefPrincipal.java
@@ -61,7 +61,7 @@ public final class AuthnContextDeclRefPrincipal implements CloneablePrincipal {
final AuthnContextDeclRef ref = (AuthnContextDeclRef) Constraint.isNotNull(
XMLObjectSupport.getBuilder(AuthnContextDeclRef.DEFAULT_ELEMENT_NAME),
"No builder for AuthnContextDeclRef").buildObject(AuthnContextDeclRef.DEFAULT_ELEMENT_NAME);
- ref.setAuthnContextDeclRef(getName());
+ ref.setURI(getName());
return ref;
}
diff --git a/idp-saml-api/src/test/java/net/shibboleth/idp/saml/authn/principal/AuthnContextClassRefPrincipalTest.java b/idp-saml-api/src/test/java/net/shibboleth/idp/saml/authn/principal/AuthnContextClassRefPrincipalTest.java
index e72549c..b074247 100644
--- a/idp-saml-api/src/test/java/net/shibboleth/idp/saml/authn/principal/AuthnContextClassRefPrincipalTest.java
+++ b/idp-saml-api/src/test/java/net/shibboleth/idp/saml/authn/principal/AuthnContextClassRefPrincipalTest.java
@@ -42,7 +42,7 @@ public class AuthnContextClassRefPrincipalTest extends XMLObjectBaseTestCase {
Assert.assertEquals(principal.getName(), AuthnContext.KERBEROS_AUTHN_CTX);
AuthnContextClassRef ref = buildXMLObject(AuthnContextClassRef.DEFAULT_ELEMENT_NAME);
- ref.setAuthnContextClassRef(AuthnContext.KERBEROS_AUTHN_CTX);
+ ref.setURI(AuthnContext.KERBEROS_AUTHN_CTX);
Element xml = getMarshaller(AuthnContextClassRef.DEFAULT_ELEMENT_NAME).marshall(ref);
assertXMLEquals(xml.getOwnerDocument(), principal.getAuthnContextClassRef());
diff --git a/idp-saml-api/src/test/java/net/shibboleth/idp/saml/authn/principal/AuthnContextDeclRefPrincipalTest.java b/idp-saml-api/src/test/java/net/shibboleth/idp/saml/authn/principal/AuthnContextDeclRefPrincipalTest.java
index bc74b00..c8b422b 100644
--- a/idp-saml-api/src/test/java/net/shibboleth/idp/saml/authn/principal/AuthnContextDeclRefPrincipalTest.java
+++ b/idp-saml-api/src/test/java/net/shibboleth/idp/saml/authn/principal/AuthnContextDeclRefPrincipalTest.java
@@ -41,7 +41,7 @@ public class AuthnContextDeclRefPrincipalTest extends XMLObjectBaseTestCase {
Assert.assertEquals(principal.getName(), "http://opensaml.org/mydecl");
AuthnContextDeclRef ref = buildXMLObject(AuthnContextDeclRef.DEFAULT_ELEMENT_NAME);
- ref.setAuthnContextDeclRef("http://opensaml.org/mydecl");
+ ref.setURI("http://opensaml.org/mydecl");
Element xml = getMarshaller(AuthnContextDeclRef.DEFAULT_ELEMENT_NAME).marshall(ref);
assertXMLEquals(xml.getOwnerDocument(), principal.getAuthnContextDeclRef());
diff --git a/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/audit/impl/AuthnContextAuditExtractor.java b/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/audit/impl/AuthnContextAuditExtractor.java
index e0bb104..7716d61 100644
--- a/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/audit/impl/AuthnContextAuditExtractor.java
+++ b/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/audit/impl/AuthnContextAuditExtractor.java
@@ -68,9 +68,9 @@ public class AuthnContextAuditExtractor implements Function<ProfileRequestContex
if (statement.getAuthnContext() != null) {
final AuthnContext ac = statement.getAuthnContext();
if (ac.getAuthnContextClassRef() != null) {
- return ac.getAuthnContextClassRef().getAuthnContextClassRef();
+ return ac.getAuthnContextClassRef().getURI();
} else if (ac.getAuthnContextDeclRef() != null) {
- return ac.getAuthnContextDeclRef().getAuthnContextDeclRef();
+ return ac.getAuthnContextDeclRef().getURI();
}
}
}
diff --git a/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/authn/principal/impl/MapDrivenAuthnContextTranslationStrategy.java b/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/authn/principal/impl/MapDrivenAuthnContextTranslationStrategy.java
index 441ede1..b836b52 100644
--- a/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/authn/principal/impl/MapDrivenAuthnContextTranslationStrategy.java
+++ b/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/authn/principal/impl/MapDrivenAuthnContextTranslationStrategy.java
@@ -79,13 +79,10 @@ public class MapDrivenAuthnContextTranslationStrategy implements Function<AuthnC
if (input != null) {
final Principal principal;
- if (input.getAuthnContextClassRef() != null
- && input.getAuthnContextClassRef().getAuthnContextClassRef() != null) {
- principal = new AuthnContextClassRefPrincipal(
- input.getAuthnContextClassRef().getAuthnContextClassRef());
- } else if (input.getAuthnContextDeclRef() != null
- && input.getAuthnContextDeclRef().getAuthnContextDeclRef() != null) {
- principal = new AuthnContextDeclRefPrincipal(input.getAuthnContextDeclRef().getAuthnContextDeclRef());
+ if (input.getAuthnContextClassRef() != null && input.getAuthnContextClassRef().getURI() != null) {
+ principal = new AuthnContextClassRefPrincipal(input.getAuthnContextClassRef().getURI());
+ } else if (input.getAuthnContextDeclRef() != null && input.getAuthnContextDeclRef().getURI() != null) {
+ principal = new AuthnContextDeclRefPrincipal(input.getAuthnContextDeclRef().getURI());
} else {
log.trace("Input AuthnContext did not contain a class or decl reference, returning nothing");
return null;
diff --git a/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/saml2/profile/impl/ProcessRequestedAuthnContext.java b/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/saml2/profile/impl/ProcessRequestedAuthnContext.java
index 0d12dc2..d56edb1 100644
--- a/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/saml2/profile/impl/ProcessRequestedAuthnContext.java
+++ b/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/saml2/profile/impl/ProcessRequestedAuthnContext.java
@@ -174,21 +174,21 @@ public class ProcessRequestedAuthnContext extends AbstractAuthenticationAction {
if (!requestedCtx.getAuthnContextClassRefs().isEmpty()) {
for (final AuthnContextClassRef ref : requestedCtx.getAuthnContextClassRefs()) {
- if (ref.getAuthnContextClassRef() != null) {
- if (!ignoredContexts.contains(ref.getAuthnContextClassRef())) {
- principals.add(new AuthnContextClassRefPrincipal(ref.getAuthnContextClassRef()));
+ if (ref.getURI() != null) {
+ if (!ignoredContexts.contains(ref.getURI())) {
+ principals.add(new AuthnContextClassRefPrincipal(ref.getURI()));
} else {
- log.info("{} Ignoring AuthnContextClassRef: {}", getLogPrefix(), ref.getAuthnContextClassRef());
+ log.info("{} Ignoring AuthnContextClassRef: {}", getLogPrefix(), ref.getURI());
}
}
}
} else if (!requestedCtx.getAuthnContextDeclRefs().isEmpty()) {
for (final AuthnContextDeclRef ref : requestedCtx.getAuthnContextDeclRefs()) {
- if (ref.getAuthnContextDeclRef() != null) {
- if (!ignoredContexts.contains(ref.getAuthnContextDeclRef())) {
- principals.add(new AuthnContextDeclRefPrincipal(ref.getAuthnContextDeclRef()));
+ if (ref.getURI() != null) {
+ if (!ignoredContexts.contains(ref.getURI())) {
+ principals.add(new AuthnContextDeclRefPrincipal(ref.getURI()));
} else {
- log.info("{} Ignoring AuthnContextDeclRef: {}", getLogPrefix(), ref.getAuthnContextDeclRef());
+ log.info("{} Ignoring AuthnContextDeclRef: {}", getLogPrefix(), ref.getURI());
}
}
}
diff --git a/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/saml2/profile/impl/ValidateSAMLAuthentication.java b/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/saml2/profile/impl/ValidateSAMLAuthentication.java
index bf221b7..391213b 100644
--- a/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/saml2/profile/impl/ValidateSAMLAuthentication.java
+++ b/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/saml2/profile/impl/ValidateSAMLAuthentication.java
@@ -311,14 +311,20 @@ public class ValidateSAMLAuthentication extends AbstractValidationAction {
translated.stream().map(Principal::getName).collect(Collectors.toUnmodifiableList()));
}
}
- } else if (authnContext.getAuthnContextClassRef() != null &&
- authnContext.getAuthnContextClassRef().getAuthnContextClassRef() != null) {
- subject.getPrincipals().add(new AuthnContextClassRefPrincipal(
- authnContext.getAuthnContextClassRef().getAuthnContextClassRef()));
- } else if (authnContext.getAuthnContextDeclRef() != null
- && authnContext.getAuthnContextDeclRef().getAuthnContextDeclRef() != null) {
- subject.getPrincipals().add(new AuthnContextDeclRefPrincipal(
- authnContext.getAuthnContextDeclRef().getAuthnContextDeclRef()));
+ } else if (authnContext.getAuthnContextClassRef() != null) {
+ final String classRef = authnContext.getAuthnContextClassRef().getURI();
+ if (classRef != null) {
+ subject.getPrincipals().add(new AuthnContextClassRefPrincipal(classRef));
+ log.debug("{} Added AuthnContextClassRef from assertion: {}", getLogPrefix(), classRef);
+ }
+ } else if (authnContext.getAuthnContextDeclRef() != null) {
+ final String declRef = authnContext.getAuthnContextDeclRef().getURI();
+ if (declRef != null) {
+ subject.getPrincipals().add(new AuthnContextDeclRefPrincipal(declRef));
+ log.debug("{} Added AuthnContextDeclRef from assertion: {}", getLogPrefix(), declRef);
+ }
+ } else {
+ log.warn("{} No AuthnContext information usable from assertion", getLogPrefix());
}
final ProxyAuthenticationPrincipal proxied = new ProxyAuthenticationPrincipal();
diff --git a/idp-saml-impl/src/test/java/net/shibboleth/idp/saml/saml2/profile/impl/AddAuthnRequestTest.java b/idp-saml-impl/src/test/java/net/shibboleth/idp/saml/saml2/profile/impl/AddAuthnRequestTest.java
index e01926a..48a73a2 100644
--- a/idp-saml-impl/src/test/java/net/shibboleth/idp/saml/saml2/profile/impl/AddAuthnRequestTest.java
+++ b/idp-saml-impl/src/test/java/net/shibboleth/idp/saml/saml2/profile/impl/AddAuthnRequestTest.java
@@ -200,8 +200,8 @@ public class AddAuthnRequestTest extends OpenSAMLInitBaseTestCase {
assertNotNull(rac);
assertEquals(rac.getComparison(), AuthnContextComparisonTypeEnumeration.EXACT);
assertEquals(rac.getAuthnContextClassRefs().size(), 2);
- assertEquals(rac.getAuthnContextClassRefs().get(0).getAuthnContextClassRef(), AuthnContext.KERBEROS_AUTHN_CTX);
- assertEquals(rac.getAuthnContextClassRefs().get(1).getAuthnContextClassRef(), AuthnContext.X509_AUTHN_CTX);
+ assertEquals(rac.getAuthnContextClassRefs().get(0).getURI(), AuthnContext.KERBEROS_AUTHN_CTX);
+ assertEquals(rac.getAuthnContextClassRefs().get(1).getURI(), AuthnContext.X509_AUTHN_CTX);
((BrowserSSOProfileConfiguration) rpc.getProfileConfig()).setAuthnContextComparison(
AuthnContextComparisonTypeEnumeration.EXACT);
@@ -218,8 +218,8 @@ public class AddAuthnRequestTest extends OpenSAMLInitBaseTestCase {
assertNotNull(rac);
assertEquals(rac.getComparison(), AuthnContextComparisonTypeEnumeration.EXACT);
assertEquals(rac.getAuthnContextClassRefs().size(), 2);
- assertEquals(rac.getAuthnContextClassRefs().get(0).getAuthnContextClassRef(), AuthnContext.KERBEROS_AUTHN_CTX);
- assertEquals(rac.getAuthnContextClassRefs().get(1).getAuthnContextClassRef(), AuthnContext.X509_AUTHN_CTX);
+ assertEquals(rac.getAuthnContextClassRefs().get(0).getURI(), AuthnContext.KERBEROS_AUTHN_CTX);
+ assertEquals(rac.getAuthnContextClassRefs().get(1).getURI(), AuthnContext.X509_AUTHN_CTX);
}
}
\ No newline at end of file
diff --git a/idp-saml-impl/src/test/java/net/shibboleth/idp/saml/saml2/profile/impl/AddAuthnStatementToAssertionTest.java b/idp-saml-impl/src/test/java/net/shibboleth/idp/saml/saml2/profile/impl/AddAuthnStatementToAssertionTest.java
index 0074194..03cde8c 100644
--- a/idp-saml-impl/src/test/java/net/shibboleth/idp/saml/saml2/profile/impl/AddAuthnStatementToAssertionTest.java
+++ b/idp-saml-impl/src/test/java/net/shibboleth/idp/saml/saml2/profile/impl/AddAuthnStatementToAssertionTest.java
@@ -156,7 +156,7 @@ public class AddAuthnStatementToAssertionTest extends OpenSAMLInitBaseTestCase {
final AuthnContext authnContext = authenticationStatement.getAuthnContext();
Assert.assertNotNull(authnContext);
Assert.assertNotNull(authnContext.getAuthnContextClassRef());
- Assert.assertEquals(authnContext.getAuthnContextClassRef().getAuthnContextClassRef(), "Test");
+ Assert.assertEquals(authnContext.getAuthnContextClassRef().getURI(), "Test");
Assert.assertTrue(authnContext.getAuthenticatingAuthorities().isEmpty());
}
@@ -208,7 +208,7 @@ public class AddAuthnStatementToAssertionTest extends OpenSAMLInitBaseTestCase {
final AuthnContext authnContext = authenticationStatement.getAuthnContext();
Assert.assertNotNull(authnContext);
Assert.assertNotNull(authnContext.getAuthnContextClassRef());
- Assert.assertEquals(authnContext.getAuthnContextClassRef().getAuthnContextClassRef(), "Bar");
+ Assert.assertEquals(authnContext.getAuthnContextClassRef().getURI(), "Bar");
Assert.assertTrue(authnContext.getAuthenticatingAuthorities().isEmpty());
}
diff --git a/idp-saml-impl/src/test/java/net/shibboleth/idp/saml/saml2/profile/impl/ProcessRequestedAuthnContextTest.java b/idp-saml-impl/src/test/java/net/shibboleth/idp/saml/saml2/profile/impl/ProcessRequestedAuthnContextTest.java
index 81fba35..c643888 100644
--- a/idp-saml-impl/src/test/java/net/shibboleth/idp/saml/saml2/profile/impl/ProcessRequestedAuthnContextTest.java
+++ b/idp-saml-impl/src/test/java/net/shibboleth/idp/saml/saml2/profile/impl/ProcessRequestedAuthnContextTest.java
@@ -114,7 +114,7 @@ public class ProcessRequestedAuthnContextTest extends OpenSAMLInitBaseTestCase {
final RequestedAuthnContext rac = racBuilder.buildObject();
((AuthnRequest) prc.getInboundMessageContext().getMessage()).setRequestedAuthnContext(rac);
final AuthnContextClassRef ref = classBuilder.buildObject();
- ref.setAuthnContextClassRef(AuthnContext.PASSWORD_AUTHN_CTX);
+ ref.setURI(AuthnContext.PASSWORD_AUTHN_CTX);
rac.getAuthnContextClassRefs().add(ref);
((BrowserSSOProfileConfiguration) prc.getSubcontext(RelyingPartyContext.class).getProfileConfig()).setDisallowedFeatures(
@@ -130,7 +130,7 @@ public class ProcessRequestedAuthnContextTest extends OpenSAMLInitBaseTestCase {
final RequestedAuthnContext rac = racBuilder.buildObject();
((AuthnRequest) prc.getInboundMessageContext().getMessage()).setRequestedAuthnContext(rac);
final AuthnContextClassRef ref = classBuilder.buildObject();
- ref.setAuthnContextClassRef(AuthnContext.UNSPECIFIED_AUTHN_CTX);
+ ref.setURI(AuthnContext.UNSPECIFIED_AUTHN_CTX);
rac.getAuthnContextClassRefs().add(ref);
((BrowserSSOProfileConfiguration) prc.getSubcontext(RelyingPartyContext.class).getProfileConfig()).setDisallowedFeatures(
@@ -146,7 +146,7 @@ public class ProcessRequestedAuthnContextTest extends OpenSAMLInitBaseTestCase {
final RequestedAuthnContext rac = racBuilder.buildObject();
((AuthnRequest) prc.getInboundMessageContext().getMessage()).setRequestedAuthnContext(rac);
final AuthnContextClassRef ref = classBuilder.buildObject();
- ref.setAuthnContextClassRef(AuthnContext.PPT_AUTHN_CTX);
+ ref.setURI(AuthnContext.PPT_AUTHN_CTX);
rac.getAuthnContextClassRefs().add(ref);
final Event event = action.execute(src);
@@ -164,10 +164,10 @@ public class ProcessRequestedAuthnContextTest extends OpenSAMLInitBaseTestCase {
((AuthnRequest) prc.getInboundMessageContext().getMessage()).setRequestedAuthnContext(rac);
rac.setComparison(AuthnContextComparisonTypeEnumeration.MINIMUM);
AuthnContextClassRef ref = classBuilder.buildObject();
- ref.setAuthnContextClassRef(AuthnContext.PPT_AUTHN_CTX);
+ ref.setURI(AuthnContext.PPT_AUTHN_CTX);
rac.getAuthnContextClassRefs().add(ref);
ref = classBuilder.buildObject();
- ref.setAuthnContextClassRef(AuthnContext.KERBEROS_AUTHN_CTX);
+ ref.setURI(AuthnContext.KERBEROS_AUTHN_CTX);
rac.getAuthnContextClassRefs().add(ref);
final Event event = action.execute(src);
@@ -184,10 +184,10 @@ public class ProcessRequestedAuthnContextTest extends OpenSAMLInitBaseTestCase {
((AuthnRequest) prc.getInboundMessageContext().getMessage()).setRequestedAuthnContext(rac);
rac.setComparison(AuthnContextComparisonTypeEnumeration.MINIMUM);
AuthnContextDeclRef ref = declBuilder.buildObject();
- ref.setAuthnContextDeclRef(AuthnContext.PPT_AUTHN_CTX);
+ ref.setURI(AuthnContext.PPT_AUTHN_CTX);
rac.getAuthnContextDeclRefs().add(ref);
ref = declBuilder.buildObject();
- ref.setAuthnContextDeclRef(AuthnContext.KERBEROS_AUTHN_CTX);
+ ref.setURI(AuthnContext.KERBEROS_AUTHN_CTX);
rac.getAuthnContextDeclRefs().add(ref);
final Event event = action.execute(src);
@@ -203,7 +203,7 @@ public class ProcessRequestedAuthnContextTest extends OpenSAMLInitBaseTestCase {
final RequestedAuthnContext rac = racBuilder.buildObject();
((AuthnRequest) prc.getInboundMessageContext().getMessage()).setRequestedAuthnContext(rac);
final AuthnContextClassRef ref = classBuilder.buildObject();
- ref.setAuthnContextClassRef(AuthnContext.UNSPECIFIED_AUTHN_CTX);
+ ref.setURI(AuthnContext.UNSPECIFIED_AUTHN_CTX);
rac.getAuthnContextClassRefs().add(ref);
final Event event = action.execute(src);
@@ -217,7 +217,7 @@ public class ProcessRequestedAuthnContextTest extends OpenSAMLInitBaseTestCase {
final RequestedAuthnContext rac = racBuilder.buildObject();
((AuthnRequest) prc.getInboundMessageContext().getMessage()).setRequestedAuthnContext(rac);
final AuthnContextClassRef ref = classBuilder.buildObject();
- ref.setAuthnContextClassRef(AuthnContext.PPT_AUTHN_CTX);
+ ref.setURI(AuthnContext.PPT_AUTHN_CTX);
rac.getAuthnContextClassRefs().add(ref);
action = new ProcessRequestedAuthnContext();
--
To stop receiving notification emails like this one, please contact
the administrator of this repository.
More information about the commits
mailing list