Changed casing on "URI" to "Url" to match other parameters.

pull/210/head
Justin Richer 2012-08-17 15:32:04 -04:00
parent d14f55004c
commit 012bb4afd7
5 changed files with 32 additions and 36 deletions

View File

@ -288,7 +288,7 @@ public class AbstractOIDCAuthenticationFilter extends
form.add("client_secret", serverConfig.getClientSecret()); form.add("client_secret", serverConfig.getClientSecret());
if (debug) { if (debug) {
logger.debug("tokenEndpointURI = " + serverConfig.getTokenEndpointURI()); logger.debug("tokenEndpointURI = " + serverConfig.getTokenEndpointUrl());
logger.debug("form = " + form); logger.debug("form = " + form);
} }
@ -296,7 +296,7 @@ public class AbstractOIDCAuthenticationFilter extends
try { try {
jsonString = restTemplate.postForObject( jsonString = restTemplate.postForObject(
serverConfig.getTokenEndpointURI(), form, String.class); serverConfig.getTokenEndpointUrl(), form, String.class);
} catch (HttpClientErrorException httpClientErrorException) { } catch (HttpClientErrorException httpClientErrorException) {
// Handle error // Handle error
@ -489,7 +489,7 @@ public class AbstractOIDCAuthenticationFilter extends
// TODO: display, prompt, request, request_uri // TODO: display, prompt, request, request_uri
String authRequest = AbstractOIDCAuthenticationFilter.buildURL(serverConfiguration.getAuthorizationEndpointURI(), urlVariables); String authRequest = AbstractOIDCAuthenticationFilter.buildURL(serverConfiguration.getAuthorizationEndpointUrl(), urlVariables);
logger.debug("Auth Request: " + authRequest); logger.debug("Auth Request: " + authRequest);

View File

@ -60,17 +60,13 @@ public class OIDCAuthenticationFilter extends AbstractOIDCAuthenticationFilter {
// Validating configuration // Validating configuration
Assert.notNull(oidcServerConfig.getAuthorizationEndpointURI(), Assert.notNull(oidcServerConfig.getAuthorizationEndpointUrl(), "An Authorization Endpoint URI must be supplied");
"An Authorization Endpoint URI must be supplied");
Assert.notNull(oidcServerConfig.getTokenEndpointURI(), Assert.notNull(oidcServerConfig.getTokenEndpointUrl(), "A Token ID Endpoint URI must be supplied");
"A Token ID Endpoint URI must be supplied");
Assert.notNull(oidcServerConfig.getClientId(), Assert.notNull(oidcServerConfig.getClientId(), "A Client ID must be supplied");
"A Client ID must be supplied");
Assert.notNull(oidcServerConfig.getClientSecret(), Assert.notNull(oidcServerConfig.getClientSecret(), "A Client Secret must be supplied");
"A Client Secret must be supplied");
} }
/* /*
@ -110,8 +106,12 @@ public class OIDCAuthenticationFilter extends AbstractOIDCAuthenticationFilter {
return null; return null;
} }
public void setAuthorizationEndpointURI(String authorizationEndpointURI) { public void setAuthorizationEndpointUrl(String authorizationEndpointUrl) {
oidcServerConfig.setAuthorizationEndpointURI(authorizationEndpointURI); oidcServerConfig.setAuthorizationEndpointUrl(authorizationEndpointUrl);
}
public void setTokenEndpointUrl(String tokenEndpointUrl) {
oidcServerConfig.setTokenEndpointUrl(tokenEndpointUrl);
} }
public void setClientId(String clientId) { public void setClientId(String clientId) {
@ -122,10 +122,6 @@ public class OIDCAuthenticationFilter extends AbstractOIDCAuthenticationFilter {
oidcServerConfig.setClientSecret(clientSecret); oidcServerConfig.setClientSecret(clientSecret);
} }
public void setTokenEndpointURI(String tokenEndpointURI) {
oidcServerConfig.setTokenEndpointURI(tokenEndpointURI);
}
public void setX509EncryptUrl(String x509EncryptUrl) { public void setX509EncryptUrl(String x509EncryptUrl) {
oidcServerConfig.setX509EncryptUrl(x509EncryptUrl); oidcServerConfig.setX509EncryptUrl(x509EncryptUrl);
} }

View File

@ -40,10 +40,10 @@ public class OIDCSignedRequestFilter extends AbstractOIDCAuthenticationFilter {
// Validating configuration // Validating configuration
Assert.notNull(oidcServerConfig.getAuthorizationEndpointURI(), Assert.notNull(oidcServerConfig.getAuthorizationEndpointUrl(),
"An Authorization Endpoint URI must be supplied"); "An Authorization Endpoint URI must be supplied");
Assert.notNull(oidcServerConfig.getTokenEndpointURI(), Assert.notNull(oidcServerConfig.getTokenEndpointUrl(),
"A Token ID Endpoint URI must be supplied"); "A Token ID Endpoint URI must be supplied");
Assert.notNull(oidcServerConfig.getClientId(), Assert.notNull(oidcServerConfig.getClientId(),
@ -93,7 +93,7 @@ public class OIDCSignedRequestFilter extends AbstractOIDCAuthenticationFilter {
urlVariables.put("request", jwt.toString()); urlVariables.put("request", jwt.toString());
String authRequest = AbstractOIDCAuthenticationFilter.buildURL(serverConfiguration.getAuthorizationEndpointURI(), urlVariables); String authRequest = AbstractOIDCAuthenticationFilter.buildURL(serverConfiguration.getAuthorizationEndpointUrl(), urlVariables);
logger.debug("Auth Request: " + authRequest); logger.debug("Auth Request: " + authRequest);
@ -152,10 +152,10 @@ public class OIDCSignedRequestFilter extends AbstractOIDCAuthenticationFilter {
/** /**
* @param authorizationEndpointURI * @param authorizationEndpointURI
* @see org.mitre.openid.connect.config.OIDCServerConfiguration#setAuthorizationEndpointURI(java.lang.String) * @see org.mitre.openid.connect.config.OIDCServerConfiguration#setAuthorizationEndpointUrl(java.lang.String)
*/ */
public void setAuthorizationEndpointURI(String authorizationEndpointURI) { public void setAuthorizationEndpointURI(String authorizationEndpointURI) {
oidcServerConfig.setAuthorizationEndpointURI(authorizationEndpointURI); oidcServerConfig.setAuthorizationEndpointUrl(authorizationEndpointURI);
} }
/** /**
@ -184,10 +184,10 @@ public class OIDCSignedRequestFilter extends AbstractOIDCAuthenticationFilter {
/** /**
* @param tokenEndpointURI * @param tokenEndpointURI
* @see org.mitre.openid.connect.config.OIDCServerConfiguration#setTokenEndpointURI(java.lang.String) * @see org.mitre.openid.connect.config.OIDCServerConfiguration#setTokenEndpointUrl(java.lang.String)
*/ */
public void setTokenEndpointURI(String tokenEndpointURI) { public void setTokenEndpointURI(String tokenEndpointURI) {
oidcServerConfig.setTokenEndpointURI(tokenEndpointURI); oidcServerConfig.setTokenEndpointUrl(tokenEndpointURI);
} }
/** /**

View File

@ -41,7 +41,7 @@ public class KeyFetcher {
try { try {
jsonString = restTemplate.getForObject( jsonString = restTemplate.getForObject(
serverConfig.getTokenEndpointURI(), String.class); serverConfig.getTokenEndpointUrl(), String.class);
} catch (HttpClientErrorException httpClientErrorException) { } catch (HttpClientErrorException httpClientErrorException) {
throw new AuthenticationServiceException( throw new AuthenticationServiceException(

View File

@ -23,9 +23,9 @@ import java.net.URI;
*/ */
public class OIDCServerConfiguration { public class OIDCServerConfiguration {
private String authorizationEndpointURI; private String authorizationEndpointUrl;
private String tokenEndpointURI; private String tokenEndpointUrl;
private String clientSecret; private String clientSecret;
@ -43,8 +43,8 @@ public class OIDCServerConfiguration {
private String userInfoUrl; private String userInfoUrl;
public String getAuthorizationEndpointURI() { public String getAuthorizationEndpointUrl() {
return authorizationEndpointURI; return authorizationEndpointUrl;
} }
public String getClientId() { public String getClientId() {
@ -59,12 +59,12 @@ public class OIDCServerConfiguration {
return clientSecret; return clientSecret;
} }
public String getTokenEndpointURI() { public String getTokenEndpointUrl() {
return tokenEndpointURI; return tokenEndpointUrl;
} }
public void setAuthorizationEndpointURI(String authorizationEndpointURI) { public void setAuthorizationEndpointUrl(String authorizationEndpointURI) {
this.authorizationEndpointURI = authorizationEndpointURI; this.authorizationEndpointUrl = authorizationEndpointURI;
} }
public void setClientId(String clientId) { public void setClientId(String clientId) {
@ -79,8 +79,8 @@ public class OIDCServerConfiguration {
this.clientSecret = clientSecret; this.clientSecret = clientSecret;
} }
public void setTokenEndpointURI(String tokenEndpointURI) { public void setTokenEndpointUrl(String tokenEndpointURI) {
this.tokenEndpointURI = tokenEndpointURI; this.tokenEndpointUrl = tokenEndpointURI;
} }
public String getX509EncryptUrl() { public String getX509EncryptUrl() {
@ -134,7 +134,7 @@ public class OIDCServerConfiguration {
*/ */
@Override @Override
public String toString() { public String toString() {
return "OIDCServerConfiguration [authorizationEndpointURI=" + authorizationEndpointURI + ", tokenEndpointURI=" + tokenEndpointURI + ", clientSecret=" + clientSecret + ", clientId=" + clientId + ", issuer=" + issuer + ", x509EncryptUrl=" + x509EncryptUrl + ", x509SigningUrl=" return "OIDCServerConfiguration [authorizationEndpointUrl=" + authorizationEndpointUrl + ", tokenEndpointUrl=" + tokenEndpointUrl + ", clientSecret=" + clientSecret + ", clientId=" + clientId + ", issuer=" + issuer + ", x509EncryptUrl=" + x509EncryptUrl + ", x509SigningUrl="
+ x509SigningUrl + ", jwkEncryptUrl=" + jwkEncryptUrl + ", jwkSigningUrl=" + jwkSigningUrl + ", userInfoUrl=" + userInfoUrl + "]"; + x509SigningUrl + ", jwkEncryptUrl=" + jwkEncryptUrl + ", jwkSigningUrl=" + jwkSigningUrl + ", userInfoUrl=" + userInfoUrl + "]";
} }