Bugfix 2.0.1 (#144)

soumame · web-flow · commit d96f9f716c4a · 2025-02-18T15:19:04.000+08:00
* chore: update project version to 2.0.1 * fix: make user creation CLI command await role setting Unable to add "admin" role for created user from cli #134 * chore: remove server telemetry configuration file * feat: add session-based WebSocket proxy upgrade handling Implement WebSocket upgrade handling with session-based validation: - Extract sessionId from the request URL - Check if sessionId exists and has a corresponding proxy - Return 404 if session is invalid - Proceed with proxy upgrade for valid sessions Server crashes when Minecraft has connected #135
diff --git a/apps/backend/package.json b/apps/backend/package.json
@@ -1,7 +1,7 @@
 {
   "name": "backend",
   "private": true,
-  "version": "2.0.0",
+  "version": "2.0.1",
   "type": "module",
   "exports": {
     "./hc": {
diff --git a/apps/backend/src/cli.ts b/apps/backend/src/cli.ts
@@ -79,9 +79,9 @@ program
 			displayName: credentials.username,
 			username: credentials.username,
 		})
-			.then((result) => {
+			.then(async (result) => {
 				console.info("User created: ", result.user);
-				setRole({ userId: result.user.id, role });
+				await setRole({ userId: result.user.id, role });
 				console.info("Role set: ", role);
 				process.exit();
 			})
diff --git a/apps/backend/src/modules/vm/index.ts b/apps/backend/src/modules/vm/index.ts
@@ -83,7 +83,17 @@ const server = serve({
 	port: vmPort,
 	createServer: createServer,
 });
+
 server.on("upgrade", (req, socket, head) => {
+	const urlParts = req.url?.split("/");
+	const sessionId = urlParts?.[1];
+
+	if (!sessionId || !vmProxies.has(sessionId)) {
+		socket.write("HTTP/1.1 404 Not Found\r\n\r\n");
+		socket.destroy();
+		return;
+	}
+
 	proxy.upgrade(req, socket as nodeSocket, head);
 });
 
diff --git a/apps/backend/src/serverTelemetry.ts b/apps/backend/src/serverTelemetry.ts
diff --git a/apps/docs/package.json b/apps/docs/package.json
@@ -1,7 +1,7 @@
 {
   "name": "docs",
   "type": "module",
-  "version": "2.0.0",
+  "version": "2.0.1",
   "scripts": {
     "dev": "astro dev",
     "start": "astro dev",
diff --git a/apps/frontend/package.json b/apps/frontend/package.json
@@ -1,7 +1,7 @@
 {
   "name": "frontend",
   "private": true,
-  "version": "2.0.0",
+  "version": "2.0.1",
   "type": "module",
   "scripts": {
     "dev": "wireit",
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "tutoriallm",
-  "version": "2.0.0",
+  "version": "2.0.1",
   "private": true,
   "type": "module",
   "packageManager": "pnpm@9.6.0",
diff --git a/packages/extensions/package.json b/packages/extensions/package.json
@@ -2,7 +2,7 @@
   "name": "extensions",
   "private": true,
   "type": "module",
-  "version": "2.0.0",
+  "version": "2.0.1",
   "exports": {
     ".": "./dist/index.js"
   },

Original file line number	Diff line number	Diff line change
`@@ -1,7 +1,7 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "backend",`
`3`	`3`	`"private": true,`
`4`		`- "version": "2.0.0",`
	`4`	`+ "version": "2.0.1",`
`5`	`5`	`"type": "module",`
`6`	`6`	`"exports": {`
`7`	`7`	`"./hc": {`
Original file line number	Diff line number	Diff line change
`@@ -1,7 +1,7 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "docs",`
`3`	`3`	`"type": "module",`
`4`		`- "version": "2.0.0",`
	`4`	`+ "version": "2.0.1",`
`5`	`5`	`"scripts": {`
`6`	`6`	`"dev": "astro dev",`
`7`	`7`	`"start": "astro dev",`
Original file line number	Diff line number	Diff line change
`@@ -1,7 +1,7 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "frontend",`
`3`	`3`	`"private": true,`
`4`		`- "version": "2.0.0",`
	`4`	`+ "version": "2.0.1",`
`5`	`5`	`"type": "module",`
`6`	`6`	`"scripts": {`
`7`	`7`	`"dev": "wireit",`
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "tutoriallm",`
`3`		`- "version": "2.0.0",`
	`3`	`+ "version": "2.0.1",`
`4`	`4`	`"private": true,`
`5`	`5`	`"type": "module",`
`6`	`6`	`"packageManager": "[email protected]",`