Merge pull request #228 from scottslewis/master

scottslewis · web-flow · commit 357794f14223 · 2025-08-18T17:53:13.000-07:00
Fix for ServerStringChannel.writeMessage to accomodate mcp notifyClients messaging before client connect
diff --git a/framework/bundles/org.eclipse.ecf.ai.mcp.transports/src/org/eclipse/ecf/ai/mcp/transports/ServerStringChannel.java b/framework/bundles/org.eclipse.ecf.ai.mcp.transports/src/org/eclipse/ecf/ai/mcp/transports/ServerStringChannel.java
@@ -83,7 +83,9 @@ public void writeMessage(String message) throws IOException {
 		if (c != null) {
 			writeMessageToChannel(c, message);
 		} else {
-			throw new IOException("not connected");
+			if (logger.isDebugEnabled()) {
+				logger.debug("No acceptedClient attached to writeMessage={}", message);
+			}
 		}
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -83,7 +83,9 @@ public void writeMessage(String message) throws IOException {`
`83`	`83`	`if (c != null) {`
`84`	`84`	`writeMessageToChannel(c, message);`
`85`	`85`	`} else {`
`86`		`- throw new IOException("not connected");`
	`86`	`+ if (logger.isDebugEnabled()) {`
	`87`	`+ logger.debug("No acceptedClient attached to writeMessage={}", message);`
	`88`	`+ }`
`87`	`89`	`}`
`88`	`90`	`}`
`89`	`91`