feat: #697 removed system property option to change default validation toggle

Author: ashakirin

mcp-core/src/main/java/io/modelcontextprotocol/server/McpServer.java

@@ -25,7 +25,6 @@
 import io.modelcontextprotocol.util.Assert;
 import io.modelcontextprotocol.util.DefaultMcpUriTemplateManagerFactory;
 import io.modelcontextprotocol.util.McpUriTemplateManagerFactory;
-import io.modelcontextprotocol.util.ToolInputValidator;
 import io.modelcontextprotocol.util.ToolNameValidator;
 import reactor.core.publisher.Mono;
 
@@ -294,7 +293,7 @@ abstract class AsyncSpecification<S extends AsyncSpecification<S>> {
 
 		boolean strictToolNameValidation = ToolNameValidator.isStrictByDefault();
 
-		boolean validateToolInputs = ToolInputValidator.isEnabledByDefault();
+		boolean validateToolInputs = true;
 
 		/**
 		 * The Model Context Protocol (MCP) allows servers to expose tools that can be
@@ -888,7 +887,7 @@ abstract class SyncSpecification<S extends SyncSpecification<S>> {
 
 		boolean strictToolNameValidation = ToolNameValidator.isStrictByDefault();
 
-		boolean validateToolInputs = ToolInputValidator.isEnabledByDefault();
+		boolean validateToolInputs = true;
 
 		/**
 		 * The Model Context Protocol (MCP) allows servers to expose tools that can be
@@ -1431,7 +1430,7 @@ class StatelessAsyncSpecification {
 
 		boolean strictToolNameValidation = ToolNameValidator.isStrictByDefault();
 
-		boolean validateToolInputs = ToolInputValidator.isEnabledByDefault();
+		boolean validateToolInputs = true;
 
 		/**
 		 * The Model Context Protocol (MCP) allows servers to expose tools that can be
@@ -1929,7 +1928,7 @@ class StatelessSyncSpecification {
 
 		boolean strictToolNameValidation = ToolNameValidator.isStrictByDefault();
 
-		boolean validateToolInputs = ToolInputValidator.isEnabledByDefault();
+		boolean validateToolInputs = true;
 
 		/**
 		 * The Model Context Protocol (MCP) allows servers to expose tools that can be

mcp-core/src/main/java/io/modelcontextprotocol/util/ToolInputValidator.java

@@ -24,23 +24,9 @@ public final class ToolInputValidator {
 
 	private static final Logger logger = LoggerFactory.getLogger(ToolInputValidator.class);
 
-	/**
-	 * System property to disable tool input validation. Set to "false" to disable.
-	 * Default is true (validation enabled).
-	 */
-	public static final String VALIDATE_TOOL_INPUTS_PROPERTY = "io.modelcontextprotocol.validateToolInputs";
-
 	private ToolInputValidator() {
 	}
 
-	/**
-	 * Returns whether validation is enabled by default based on system property.
-	 * @return true if validation is enabled (default), false if disabled
-	 */
-	public static boolean isEnabledByDefault() {
-		return !"false".equalsIgnoreCase(System.getProperty(VALIDATE_TOOL_INPUTS_PROPERTY));
-	}
-
 	/**
 	 * Validates tool arguments against the tool's input schema.
 	 * @param tool the tool definition containing the input schema