code review

springdoc · Mar 9, 2023 · f0be53e · f0be53e
1 parent a6ea996
commit f0be53e
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/...napi-security/src/main/java/org/springdoc/security/SpringDocSecurityOAuth2Customizer.java b/...napi-security/src/main/java/org/springdoc/security/SpringDocSecurityOAuth2Customizer.java
@@ -143,7 +143,7 @@ private void getOAuth2AuthorizationServerMetadataEndpoint(OpenAPI openAPI, Secur
 		Object oAuth2EndpointFilter =
 				new SpringDocSecurityOAuth2EndpointUtils(OAuth2AuthorizationServerMetadataEndpointFilter.class).findEndpoint(securityFilterChain);
 		if (oAuth2EndpointFilter != null) {
-			ApiResponses apiResponses = buildApiResponses(SpringDocAnnotationsUtils.resolveSchemaFromType(OAuth2AuthorizationServerMetadata.class, openAPI.getComponents(), null), openAPI);
+			ApiResponses apiResponses = buildApiResponses(SpringDocAnnotationsUtils.resolveSchemaFromType(OAuth2AuthorizationServerMetadata.class, openAPI.getComponents(), null));
 			Operation operation = buildOperation(apiResponses);
 			buildPath(oAuth2EndpointFilter, "requestMatcher", openAPI, operation, HttpMethod.GET);
 		}
@@ -159,7 +159,7 @@ private void getNimbusJwkSetEndpoint(OpenAPI openAPI, SecurityFilterChain securi
 		Object oAuth2EndpointFilter =
 				new SpringDocSecurityOAuth2EndpointUtils(NimbusJwkSetEndpointFilter.class).findEndpoint(securityFilterChain);
 		if (oAuth2EndpointFilter != null) {
-			ApiResponses apiResponses = buildApiResponses(SpringDocAnnotationsUtils.resolveSchemaFromType(JWKSet.class, openAPI.getComponents(), null), openAPI);
+			ApiResponses apiResponses = buildApiResponses(SpringDocAnnotationsUtils.resolveSchemaFromType(JWKSet.class, openAPI.getComponents(), null));
 			Operation operation = buildOperation(apiResponses);
 			operation.responses(apiResponses);
 			buildPath(oAuth2EndpointFilter, "requestMatcher", openAPI, operation, HttpMethod.GET);
@@ -224,10 +224,9 @@ private Operation buildOperation(ApiResponses apiResponses) {
 	 * Build api responses api responses.
 	 *
 	 * @param schema the schema
-	 * @param openAPI the open api
 	 * @return the api responses
 	 */
-	private ApiResponses buildApiResponses(Schema schema, OpenAPI openAPI) {
+	private ApiResponses buildApiResponses(Schema schema) {
 		ApiResponses apiResponses = new ApiResponses();
 		ApiResponse response = new ApiResponse().description(HttpStatus.OK.getReasonPhrase()).content(new Content().addMediaType(
 				APPLICATION_JSON_VALUE,
@@ -245,7 +244,7 @@ private ApiResponses buildApiResponses(Schema schema, OpenAPI openAPI) {
 	 * @return the api responses
 	 */
 	private ApiResponses buildApiResponsesWithBadRequest(Schema schema, OpenAPI openAPI) {
-		ApiResponses apiResponses = buildApiResponses(schema, openAPI);
+		ApiResponses apiResponses = buildApiResponses(schema);
 		buildOAuth2Error(openAPI, apiResponses, HttpStatus.BAD_REQUEST);
 		return apiResponses;
 	}