Skip to content

Commit 66bc5a6

Browse files
committed
Typos, renames, etc
1 parent 6d5350d commit 66bc5a6

File tree

4 files changed

+10
-7
lines changed

4 files changed

+10
-7
lines changed

oauth2-authorization-server/src/main/java/org/springframework/security/config/annotation/web/configurers/oauth2/server/authorization/OAuth2AuthorizationServerConfigurer.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,7 @@ public final class OAuth2AuthorizationServerConfigurer<B extends HttpSecurityBui
8282
OAuth2TokenRevocationEndpointFilter.DEFAULT_TOKEN_REVOCATION_ENDPOINT_URI, HttpMethod.POST.name());
8383
private final RequestMatcher jwkSetEndpointMatcher = new AntPathRequestMatcher(
8484
JwkSetEndpointFilter.DEFAULT_JWK_SET_ENDPOINT_URI, HttpMethod.GET.name());
85-
private final RequestMatcher oidcProviderMetadataEndpointMatcher = new AntPathRequestMatcher(
85+
private final RequestMatcher oidcProviderConfigurationEndpointMatcher = new AntPathRequestMatcher(
8686
OidcProviderConfigurationEndpointFilter.DEFAULT_OIDC_PROVIDER_CONFIGURATION_ENDPOINT_URI, HttpMethod.GET.name());
8787

8888
/**
@@ -141,7 +141,7 @@ public OAuth2AuthorizationServerConfigurer<B> providerSettings(ProviderSettings
141141
public List<RequestMatcher> getEndpointMatchers() {
142142
// TODO: use ProviderSettings instead
143143
return Arrays.asList(this.authorizationEndpointMatcher, this.tokenEndpointMatcher,
144-
this.tokenRevocationEndpointMatcher, this.jwkSetEndpointMatcher, this.oidcProviderMetadataEndpointMatcher);
144+
this.tokenRevocationEndpointMatcher, this.jwkSetEndpointMatcher, this.oidcProviderConfigurationEndpointMatcher);
145145
}
146146

147147
@Override

oauth2-authorization-server/src/main/java/org/springframework/security/oauth2/core/oidc/OidcProviderConfiguration.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
*/
1616
package org.springframework.security.oauth2.core.oidc;
1717

18+
import org.springframework.security.oauth2.server.authorization.Version;
1819
import org.springframework.util.Assert;
1920

2021
import java.io.Serializable;
@@ -39,6 +40,8 @@
3940
* @see <a target="_blank" href="https://openid.net/specs/openid-connect-discovery-1_0.html#ProviderConfigurationResponse">4.2. OpenID Provider Configuration Response</a>
4041
*/
4142
public class OidcProviderConfiguration implements OidcProviderMetadataClaimAccessor, Serializable {
43+
private static final long serialVersionUID = Version.SERIAL_VERSION_UID;
44+
4245
private final Map<String, Object> claims;
4346

4447
private OidcProviderConfiguration(Map<String, Object> claims) {

oauth2-authorization-server/src/main/java/org/springframework/security/oauth2/core/oidc/OidcProviderMetadataClaimNames.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121
*
2222
* @author Daniel Garnier-Moiroux
2323
* @since 0.1.0
24-
* @see <a target="_blank" href="https://openid.net/specs/openid-connect-discovery-1_0.html#ProviderMetadata">URL the OpenID Provider asserts as its Issuer identifier</a>
24+
* @see <a target="_blank" href="https://openid.net/specs/openid-connect-discovery-1_0.html#ProviderMetadata">3. OpenID Provider Metadata</a>
2525
*/
2626
public interface OidcProviderMetadataClaimNames {
2727

oauth2-authorization-server/src/main/java/org/springframework/security/oauth2/server/authorization/web/OidcProviderConfigurationEndpointFilter.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -75,10 +75,10 @@ protected void doFilterInternal(HttpServletRequest request, HttpServletResponse
7575
}
7676

7777
OidcProviderConfiguration providerConfiguration = OidcProviderConfiguration.withClaims()
78-
.issuer(providerSettings.issuer())
79-
.authorizationEndpoint(asUrl(providerSettings.issuer(), providerSettings.authorizationEndpoint()))
80-
.tokenEndpoint(asUrl(providerSettings.issuer(), providerSettings.tokenEndpoint()))
81-
.jwksUri(asUrl(providerSettings.issuer(), providerSettings.jwkSetEndpoint()))
78+
.issuer(this.providerSettings.issuer())
79+
.authorizationEndpoint(asUrl(this.providerSettings.issuer(), this.providerSettings.authorizationEndpoint()))
80+
.tokenEndpoint(asUrl(this.providerSettings.issuer(), this.providerSettings.tokenEndpoint()))
81+
.jwksUri(asUrl(this.providerSettings.issuer(), this.providerSettings.jwkSetEndpoint()))
8282
.subjectType("public")
8383
.responseType(OAuth2AuthorizationResponseType.CODE.getValue())
8484
.scope(OidcScopes.OPENID)

0 commit comments

Comments
 (0)