Warning messages when docs are explicitly enabled. Fixes #3131

bnasslahsen · bnasslahsen · commit 95c3b87318b4 · 2025-11-11T14:44:52.000+01:00
diff --git a/springdoc-openapi-starter-common/src/main/java/org/springdoc/core/configuration/SpringDocConfiguration.java b/springdoc-openapi-starter-common/src/main/java/org/springdoc/core/configuration/SpringDocConfiguration.java
@@ -736,6 +736,6 @@ MethodParameterPojoExtractor methodParameterPojoExtractor(SchemaUtils schemaUtil
 	@ConditionalOnMissingBean(name = "springDocAppInitializer")
 	@Lazy(false)
 	SpringDocAppInitializer springDocAppInitializer(SpringDocConfigProperties springDocConfigProperties){
-		return new SpringDocAppInitializer(springDocConfigProperties.getApiDocs().getPath(), SPRINGDOC_ENABLED);
+		return new SpringDocAppInitializer(springDocConfigProperties.getApiDocs().getPath(), SPRINGDOC_ENABLED, springDocConfigProperties.getApiDocs().isEnabled());
 	}
 }
diff --git a/springdoc-openapi-starter-common/src/main/java/org/springdoc/core/events/SpringDocAppInitializer.java b/springdoc-openapi-starter-common/src/main/java/org/springdoc/core/events/SpringDocAppInitializer.java
@@ -49,6 +49,11 @@ public class SpringDocAppInitializer {
 	 */
 	private final String property;
 
+	/**
+	 * The Springdoc enabled.
+	 */
+	private final boolean springdocEnabled;
+	
 	/**
 	 * The constant LOGGER.
 	 */
@@ -57,19 +62,22 @@ public class SpringDocAppInitializer {
 	/**
 	 * Instantiates a new Spring doc app initializer.
 	 *
-	 * @param endpoint the endpoint
-	 * @param property the property
+	 * @param endpoint         the endpoint
+	 * @param property         the property
+	 * @param springdocEnabled the springdoc enabled
 	 */
-	public SpringDocAppInitializer(String endpoint, String property) {
+	public SpringDocAppInitializer(String endpoint, String property, boolean springdocEnabled) {
 		this.endpoint = endpoint;
 		this.property = property;
+		this.springdocEnabled = springdocEnabled;
 	}
 
 	/**
 	 * Init.
 	 */
 	@EventListener(ApplicationReadyEvent.class)
 	public void init() {
-		LOGGER.warn("SpringDoc {} endpoint is enabled by default. To disable it in production, set the property '{}=false'", endpoint, property);
+		if(!this.springdocEnabled)
+			LOGGER.warn("SpringDoc {} endpoint is enabled by default. To disable it in production, set the property '{}=false'", endpoint, property);
 	}
 }
diff --git a/springdoc-openapi-starter-common/src/main/java/org/springdoc/core/properties/SpringDocConfigProperties.java b/springdoc-openapi-starter-common/src/main/java/org/springdoc/core/properties/SpringDocConfigProperties.java
@@ -1434,7 +1434,7 @@ public static class ApiDocs {
 		/**
 		 * Whether to generate and serve an OpenAPI document.
 		 */
-		private boolean enabled = true;
+		private boolean enabled;
 
 		/**
 		 * The Resolve schema properties.
diff --git a/springdoc-openapi-starter-common/src/main/java/org/springdoc/core/properties/SwaggerUiConfigProperties.java b/springdoc-openapi-starter-common/src/main/java/org/springdoc/core/properties/SwaggerUiConfigProperties.java
@@ -96,7 +96,7 @@ public class SwaggerUiConfigProperties extends AbstractSwaggerUiConfigProperties
 	/**
 	 * Whether to generate and serve an OpenAPI document.
 	 */
-	private boolean enabled = true;
+	private boolean enabled;
 
 	/**
 	 * The Use root path.
diff --git a/springdoc-openapi-starter-webflux-scalar/src/main/java/org/springdoc/webflux/scalar/ScalarConfiguration.java b/springdoc-openapi-starter-webflux-scalar/src/main/java/org/springdoc/webflux/scalar/ScalarConfiguration.java
@@ -102,7 +102,7 @@ ForwardedHeaderTransformer forwardedHeaderTransformer() {
 	@ConditionalOnProperty(name = SPRINGDOC_USE_MANAGEMENT_PORT, havingValue = "false", matchIfMissing = true)
 	@Lazy(false)
 	SpringDocAppInitializer springDocScalarInitializer(ScalarProperties scalarProperties){
-		return new SpringDocAppInitializer(scalarProperties.getPath(), SCALAR_ENABLED);
+		return new SpringDocAppInitializer(scalarProperties.getPath(), SCALAR_ENABLED, scalarProperties.isEnabled());
 	}
 
 	/**
@@ -135,8 +135,8 @@ ScalarActuatorController scalarActuatorController(ScalarProperties properties,
 		@Bean
 		@ConditionalOnMissingBean(name = "springDocScalarInitializer")
 		@Lazy(false)
-		SpringDocAppInitializer springDocScalarInitializer(){
-			return new SpringDocAppInitializer(DEFAULT_SCALAR_ACTUATOR_PATH, SCALAR_ENABLED);
+		SpringDocAppInitializer springDocScalarInitializer(ScalarProperties scalarProperties){
+			return new SpringDocAppInitializer(DEFAULT_SCALAR_ACTUATOR_PATH, SCALAR_ENABLED, scalarProperties.isEnabled());
 		}
 	}
 
diff --git a/springdoc-openapi-starter-webflux-ui/src/main/java/org/springdoc/webflux/ui/SwaggerConfig.java b/springdoc-openapi-starter-webflux-ui/src/main/java/org/springdoc/webflux/ui/SwaggerConfig.java
@@ -196,7 +196,7 @@ SwaggerResourceResolver swaggerResourceResolver(SwaggerUiConfigProperties swagge
 	@ConditionalOnProperty(name = SPRINGDOC_USE_MANAGEMENT_PORT, havingValue = "false", matchIfMissing = true)
 	@Lazy(false)
 	SpringDocAppInitializer springDocSwaggerInitializer(SwaggerUiConfigProperties swaggerUiConfigProperties) {
-		return new SpringDocAppInitializer(swaggerUiConfigProperties.getPath(), SPRINGDOC_SWAGGER_UI_ENABLED);
+		return new SpringDocAppInitializer(swaggerUiConfigProperties.getPath(), SPRINGDOC_SWAGGER_UI_ENABLED, swaggerUiConfigProperties.isEnabled());
 	}
 	
 	/**
@@ -233,8 +233,8 @@ SwaggerWelcomeActuator swaggerActuatorWelcome(SwaggerUiConfigProperties swaggerU
 		@Bean
 		@ConditionalOnMissingBean(name = "springDocSwaggerInitializer")
 		@Lazy(false)
-		SpringDocAppInitializer springDocSwaggerInitializer() {
-			return new SpringDocAppInitializer(DEFAULT_SWAGGER_UI_ACTUATOR_PATH, SPRINGDOC_SWAGGER_UI_ENABLED);
+		SpringDocAppInitializer springDocSwaggerInitializer(SwaggerUiConfigProperties swaggerUiConfigProperties) {
+			return new SpringDocAppInitializer(DEFAULT_SWAGGER_UI_ACTUATOR_PATH, SPRINGDOC_SWAGGER_UI_ENABLED, swaggerUiConfigProperties.isEnabled());
 		}
 	}
 }
diff --git a/springdoc-openapi-starter-webmvc-scalar/src/main/java/org/springdoc/webmvc/scalar/ScalarConfiguration.java b/springdoc-openapi-starter-webmvc-scalar/src/main/java/org/springdoc/webmvc/scalar/ScalarConfiguration.java
@@ -103,7 +103,7 @@ public FilterRegistrationBean<ForwardedHeaderFilter> forwardedHeaderFilter() {
 	@ConditionalOnProperty(name = SPRINGDOC_USE_MANAGEMENT_PORT, havingValue = "false", matchIfMissing = true)
 	@Lazy(false)
 	SpringDocAppInitializer springDocScalarInitializer(ScalarProperties scalarProperties) {
-		return new SpringDocAppInitializer(scalarProperties.getPath(), SCALAR_ENABLED);
+		return new SpringDocAppInitializer(scalarProperties.getPath(), SCALAR_ENABLED, scalarProperties.isEnabled());
 	}
 	
 	/**
@@ -137,8 +137,8 @@ ScalarActuatorController scalarActuatorController(ScalarProperties properties, W
 		@Bean
 		@ConditionalOnMissingBean(name = "springDocScalarInitializer")
 		@Lazy(false)
-		SpringDocAppInitializer springDocScalarInitializer() {
-			return new SpringDocAppInitializer(DEFAULT_SCALAR_ACTUATOR_PATH, SCALAR_ENABLED);
+		SpringDocAppInitializer springDocScalarInitializer(ScalarProperties properties) {
+			return new SpringDocAppInitializer(DEFAULT_SCALAR_ACTUATOR_PATH, SCALAR_ENABLED, properties.isEnabled());
 		}
 	}
 
diff --git a/springdoc-openapi-starter-webmvc-ui/src/main/java/org/springdoc/webmvc/ui/SwaggerConfig.java b/springdoc-openapi-starter-webmvc-ui/src/main/java/org/springdoc/webmvc/ui/SwaggerConfig.java
@@ -185,7 +185,7 @@ SwaggerResourceResolver swaggerResourceResolver(SwaggerUiConfigProperties swagge
 	@ConditionalOnProperty(name = SPRINGDOC_USE_MANAGEMENT_PORT, havingValue = "false", matchIfMissing = true)
 	@Lazy(false)
 	SpringDocAppInitializer springDocSwaggerInitializer(SwaggerUiConfigProperties swaggerUiConfigProperties) {
-		return new SpringDocAppInitializer(swaggerUiConfigProperties.getPath(), SPRINGDOC_SWAGGER_UI_ENABLED);
+		return new SpringDocAppInitializer(swaggerUiConfigProperties.getPath(), SPRINGDOC_SWAGGER_UI_ENABLED, swaggerUiConfigProperties.isEnabled());
 	}
 	
 	/**
@@ -219,8 +219,8 @@ SwaggerWelcomeActuator swaggerActuatorWelcome(SwaggerUiConfigProperties swaggerU
 		@Bean
 		@ConditionalOnMissingBean(name = "springDocSwaggerInitializer")
 		@Lazy(false)
-		SpringDocAppInitializer springDocSwaggerInitializer() {
-			return new SpringDocAppInitializer(DEFAULT_SWAGGER_UI_ACTUATOR_PATH, SPRINGDOC_SWAGGER_UI_ENABLED);
+		SpringDocAppInitializer springDocSwaggerInitializer(SwaggerUiConfigProperties swaggerUiConfigProperties) {
+			return new SpringDocAppInitializer(DEFAULT_SWAGGER_UI_ACTUATOR_PATH, SPRINGDOC_SWAGGER_UI_ENABLED, swaggerUiConfigProperties.isEnabled());
 		}
 	}
 }

Original file line number	Diff line number	Diff line change
`@@ -736,6 +736,6 @@ MethodParameterPojoExtractor methodParameterPojoExtractor(SchemaUtils schemaUtil`
`736`	`736`	`@ConditionalOnMissingBean(name = "springDocAppInitializer")`
`737`	`737`	`@Lazy(false)`
`738`	`738`	`SpringDocAppInitializer springDocAppInitializer(SpringDocConfigProperties springDocConfigProperties){`
`739`		`- return new SpringDocAppInitializer(springDocConfigProperties.getApiDocs().getPath(), SPRINGDOC_ENABLED);`
	`739`	`+ return new SpringDocAppInitializer(springDocConfigProperties.getApiDocs().getPath(), SPRINGDOC_ENABLED, springDocConfigProperties.getApiDocs().isEnabled());`
`740`	`740`	`}`
`741`	`741`	`}`
Original file line number	Diff line number	Diff line change
`@@ -102,7 +102,7 @@ ForwardedHeaderTransformer forwardedHeaderTransformer() {`
`102`	`102`	`@ConditionalOnProperty(name = SPRINGDOC_USE_MANAGEMENT_PORT, havingValue = "false", matchIfMissing = true)`
`103`	`103`	`@Lazy(false)`
`104`	`104`	`SpringDocAppInitializer springDocScalarInitializer(ScalarProperties scalarProperties){`
`105`		`- return new SpringDocAppInitializer(scalarProperties.getPath(), SCALAR_ENABLED);`
	`105`	`+ return new SpringDocAppInitializer(scalarProperties.getPath(), SCALAR_ENABLED, scalarProperties.isEnabled());`
`106`	`106`	`}`
`107`	`107`
`108`	`108`	`/**`
`@@ -135,8 +135,8 @@ ScalarActuatorController scalarActuatorController(ScalarProperties properties,`
`135`	`135`	`@Bean`
`136`	`136`	`@ConditionalOnMissingBean(name = "springDocScalarInitializer")`
`137`	`137`	`@Lazy(false)`
`138`		`- SpringDocAppInitializer springDocScalarInitializer(){`
`139`		`- return new SpringDocAppInitializer(DEFAULT_SCALAR_ACTUATOR_PATH, SCALAR_ENABLED);`
	`138`	`+ SpringDocAppInitializer springDocScalarInitializer(ScalarProperties scalarProperties){`
	`139`	`+ return new SpringDocAppInitializer(DEFAULT_SCALAR_ACTUATOR_PATH, SCALAR_ENABLED, scalarProperties.isEnabled());`
`140`	`140`	`}`
`141`	`141`	`}`
`142`	`142`
Original file line number	Diff line number	Diff line change
`@@ -196,7 +196,7 @@ SwaggerResourceResolver swaggerResourceResolver(SwaggerUiConfigProperties swagge`
`196`	`196`	`@ConditionalOnProperty(name = SPRINGDOC_USE_MANAGEMENT_PORT, havingValue = "false", matchIfMissing = true)`
`197`	`197`	`@Lazy(false)`
`198`	`198`	`SpringDocAppInitializer springDocSwaggerInitializer(SwaggerUiConfigProperties swaggerUiConfigProperties) {`
`199`		`- return new SpringDocAppInitializer(swaggerUiConfigProperties.getPath(), SPRINGDOC_SWAGGER_UI_ENABLED);`
	`199`	`+ return new SpringDocAppInitializer(swaggerUiConfigProperties.getPath(), SPRINGDOC_SWAGGER_UI_ENABLED, swaggerUiConfigProperties.isEnabled());`
`200`	`200`	`}`
`201`	`201`
`202`	`202`	`/**`
`@@ -233,8 +233,8 @@ SwaggerWelcomeActuator swaggerActuatorWelcome(SwaggerUiConfigProperties swaggerU`
`233`	`233`	`@Bean`
`234`	`234`	`@ConditionalOnMissingBean(name = "springDocSwaggerInitializer")`
`235`	`235`	`@Lazy(false)`
`236`		`- SpringDocAppInitializer springDocSwaggerInitializer() {`
`237`		`- return new SpringDocAppInitializer(DEFAULT_SWAGGER_UI_ACTUATOR_PATH, SPRINGDOC_SWAGGER_UI_ENABLED);`
	`236`	`+ SpringDocAppInitializer springDocSwaggerInitializer(SwaggerUiConfigProperties swaggerUiConfigProperties) {`
	`237`	`+ return new SpringDocAppInitializer(DEFAULT_SWAGGER_UI_ACTUATOR_PATH, SPRINGDOC_SWAGGER_UI_ENABLED, swaggerUiConfigProperties.isEnabled());`
`238`	`238`	`}`
`239`	`239`	`}`
`240`	`240`	`}`
Original file line number	Diff line number	Diff line change
`@@ -103,7 +103,7 @@ public FilterRegistrationBean<ForwardedHeaderFilter> forwardedHeaderFilter() {`
`103`	`103`	`@ConditionalOnProperty(name = SPRINGDOC_USE_MANAGEMENT_PORT, havingValue = "false", matchIfMissing = true)`
`104`	`104`	`@Lazy(false)`
`105`	`105`	`SpringDocAppInitializer springDocScalarInitializer(ScalarProperties scalarProperties) {`
`106`		`- return new SpringDocAppInitializer(scalarProperties.getPath(), SCALAR_ENABLED);`
	`106`	`+ return new SpringDocAppInitializer(scalarProperties.getPath(), SCALAR_ENABLED, scalarProperties.isEnabled());`
`107`	`107`	`}`
`108`	`108`
`109`	`109`	`/**`
`@@ -137,8 +137,8 @@ ScalarActuatorController scalarActuatorController(ScalarProperties properties, W`
`137`	`137`	`@Bean`
`138`	`138`	`@ConditionalOnMissingBean(name = "springDocScalarInitializer")`
`139`	`139`	`@Lazy(false)`
`140`		`- SpringDocAppInitializer springDocScalarInitializer() {`
`141`		`- return new SpringDocAppInitializer(DEFAULT_SCALAR_ACTUATOR_PATH, SCALAR_ENABLED);`
	`140`	`+ SpringDocAppInitializer springDocScalarInitializer(ScalarProperties properties) {`
	`141`	`+ return new SpringDocAppInitializer(DEFAULT_SCALAR_ACTUATOR_PATH, SCALAR_ENABLED, properties.isEnabled());`
`142`	`142`	`}`
`143`	`143`	`}`
`144`	`144`
Original file line number	Diff line number	Diff line change
`@@ -185,7 +185,7 @@ SwaggerResourceResolver swaggerResourceResolver(SwaggerUiConfigProperties swagge`
`185`	`185`	`@ConditionalOnProperty(name = SPRINGDOC_USE_MANAGEMENT_PORT, havingValue = "false", matchIfMissing = true)`
`186`	`186`	`@Lazy(false)`
`187`	`187`	`SpringDocAppInitializer springDocSwaggerInitializer(SwaggerUiConfigProperties swaggerUiConfigProperties) {`
`188`		`- return new SpringDocAppInitializer(swaggerUiConfigProperties.getPath(), SPRINGDOC_SWAGGER_UI_ENABLED);`
	`188`	`+ return new SpringDocAppInitializer(swaggerUiConfigProperties.getPath(), SPRINGDOC_SWAGGER_UI_ENABLED, swaggerUiConfigProperties.isEnabled());`
`189`	`189`	`}`
`190`	`190`
`191`	`191`	`/**`
`@@ -219,8 +219,8 @@ SwaggerWelcomeActuator swaggerActuatorWelcome(SwaggerUiConfigProperties swaggerU`
`219`	`219`	`@Bean`
`220`	`220`	`@ConditionalOnMissingBean(name = "springDocSwaggerInitializer")`
`221`	`221`	`@Lazy(false)`
`222`		`- SpringDocAppInitializer springDocSwaggerInitializer() {`
`223`		`- return new SpringDocAppInitializer(DEFAULT_SWAGGER_UI_ACTUATOR_PATH, SPRINGDOC_SWAGGER_UI_ENABLED);`
	`222`	`+ SpringDocAppInitializer springDocSwaggerInitializer(SwaggerUiConfigProperties swaggerUiConfigProperties) {`
	`223`	`+ return new SpringDocAppInitializer(DEFAULT_SWAGGER_UI_ACTUATOR_PATH, SPRINGDOC_SWAGGER_UI_ENABLED, swaggerUiConfigProperties.isEnabled());`
`224`	`224`	`}`
`225`	`225`	`}`
`226`	`226`	`}`