rabbit fixes

Author: jimaek

src/app.ts

@@ -51,10 +51,12 @@ app.get("/", async (_c) => {
 
 // Serve Gemini extension manifest
 app.get("/gemini-extension.json", (c) => {
+	c.header("Cache-Control", "public, max-age=3600");
 	return c.json(geminiExtension);
 });
 
 app.get("/.well-known/gemini-extension.json", (c) => {
+	c.header("Cache-Control", "public, max-age=3600");
 	return c.json(geminiExtension);
 });
 

src/lib/security.ts

@@ -67,6 +67,15 @@ export function validateHost(host: string | null): boolean {
 			return false;
 		}
 
+		// Validate suffix after closing bracket
+		const suffix = host.substring(closeBracketIndex + 1);
+		if (suffix !== "") {
+			// If there's a suffix, it must be :port (colon followed by digits)
+			if (!/^:\d+$/.test(suffix)) {
+				return false;
+			}
+		}
+
 		// Extract IPv6 address with brackets (e.g., "[::1]" from "[::1]:8080")
 		// Preserve the brackets as getAllowedHostnames returns IPv6 with brackets
 		normalizedHost = host.substring(0, closeBracketIndex + 1);

src/mcp/helpers.ts

@@ -136,7 +136,7 @@ export function formatMeasurementSummary(measurement: any): string {
 					const avg = testResult.stats.avg !== undefined ? testResult.stats.avg : "N/A";
 					const max = testResult.stats.max !== undefined ? testResult.stats.max : "N/A";
 					const loss =
-						testResult.stats.loss !== undefined
+						typeof testResult.stats.loss === "number" && !isNaN(testResult.stats.loss)
 							? testResult.stats.loss.toFixed(2)
 							: "N/A";
 

src/mcp/tools.ts

@@ -350,7 +350,8 @@ export function registerGlobalpingTools(agent: GlobalpingMCP, getToken: () => st
 			inputSchema: {
 				target: z
 					.string()
-					.describe("Domain name or IP to test (e.g., 'google.com', '1.1.1.1')"),
+					.min(1)
+					.describe("Destination hostname or IP to run the MTR against"),
 				locations: z
 					.union([z.array(z.string()), z.string()])
 					.optional()
@@ -556,7 +557,7 @@ export function registerGlobalpingTools(agent: GlobalpingMCP, getToken: () => st
 		{
 			title: "List Probe Locations",
 			description:
-				"Retrieve the list of available Globalping probe locations. Use this tool to find specific countries, cities, or ASNs to use as the 'locations' argument in other measurement tools. Avoid using this tool unless absolutely necessary, instead simplify provide the location you need to the tools above, the field is smart and will auto select the right probes.",
+				"Retrieve the list of available Globalping probe locations. Use this tool to find specific countries, cities, or ASNs to use as the 'locations' argument in other measurement tools. Avoid using this tool unless absolutely necessary, instead simply provide the location you need to the tools above, the field is smart and will auto select the right probes.",
 			annotations: {
 				readOnlyHint: true,
 			},