Skip to content

Commit 8560b19

Browse files
committed
Fix postProcessXml overriding string parameter name in test and javadoc
1 parent b6f8ad3 commit 8560b19

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

core/src/main/java/com/onelogin/saml2/authn/AuthnRequest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -119,7 +119,7 @@ public AuthnRequest(Saml2Settings settings, AuthnRequestParams params) {
119119
LOGGER.debug("AuthNRequest --> " + authnRequestString);
120120
}
121121

122-
/*
122+
/**
123123
* Allows for an extension class to post-process the AuthnRequest XML generated
124124
* for this request, in order to customize the result.
125125
* <p>

core/src/test/java/com/onelogin/saml2/test/authn/AuthnRequestTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -524,8 +524,8 @@ public void testPostProcessXml() throws Exception {
524524
Saml2Settings settings = new SettingsBuilder().fromFile("config/config.min.properties").build();
525525
AuthnRequest authnRequest = new AuthnRequest(settings) {
526526
@Override
527-
protected String postProcessXml(String authRequestXml, AuthnRequestParams params, Saml2Settings sett) {
528-
assertEquals(authRequestXml, super.postProcessXml(authRequestXml, params, sett));
527+
protected String postProcessXml(String authnRequestXml, AuthnRequestParams params, Saml2Settings sett) {
528+
assertEquals(authnRequestXml, super.postProcessXml(authnRequestXml, params, sett));
529529
assertSame(settings, sett);
530530
return "changed";
531531
}

0 commit comments

Comments
 (0)