From 157f5c3db81abf1cb2d09509ba165ff716ac4ff0 Mon Sep 17 00:00:00 2001 From: Shinigami Date: Fri, 23 Jan 2026 20:12:38 +0530 Subject: [PATCH] Add null validation to DockerMcpGatewayContainer public APIs --- .../containers/DockerMcpGatewayContainer.java | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/core/src/main/java/org/testcontainers/containers/DockerMcpGatewayContainer.java b/core/src/main/java/org/testcontainers/containers/DockerMcpGatewayContainer.java index 40e80ae2061..b7b1c0aa026 100644 --- a/core/src/main/java/org/testcontainers/containers/DockerMcpGatewayContainer.java +++ b/core/src/main/java/org/testcontainers/containers/DockerMcpGatewayContainer.java @@ -10,6 +10,8 @@ import java.util.HashMap; import java.util.List; import java.util.Map; +import java.util.Objects; + /** * Testcontainers implementation of the Docker MCP Gateway container. @@ -78,23 +80,34 @@ protected void containerIsCreated(String containerId) { } public DockerMcpGatewayContainer withServer(String server, List tools) { + Objects.requireNonNull(tools, "tools list must not be null"); + Objects.requireNonNull(server, "server must not be null"); + this.servers.add(server); this.tools.addAll(tools); return this; } public DockerMcpGatewayContainer withServer(String server, String... tools) { + Objects.requireNonNull(tools, "tools array must not be null"); + Objects.requireNonNull(server, "server must not be null"); + this.servers.add(server); this.tools.addAll(Arrays.asList(tools)); return this; } public DockerMcpGatewayContainer withSecrets(Map secrets) { + Objects.requireNonNull(secrets, "secrets map must not be null"); + this.secrets.putAll(secrets); return this; } public DockerMcpGatewayContainer withSecret(String secretKey, String secretValue) { + Objects.requireNonNull(secretKey, "secret key must not be null"); + Objects.requireNonNull(secretValue, "secret value must not be null"); + this.secrets.put(secretKey, secretValue); return this; }