Merge pull request #247 from Zekiah-A/main

Major server code cleanups, security enhancements + RELICENSE

.eslintrc.json

@@ -35,9 +35,11 @@
         "no-process-exit": "off",
         "no-unused-vars": "off",
         "prefer-const": "warn",
-        "no-const-assign": "error"
+        "no-const-assign": "error",
+        "require-jsdoc": "off",
+        "no-explicit-any": "off"
     },
     "globals": {
-        "Bun": false
+        "Bun": true
     }
-}
+}

README.md

@@ -20,7 +20,8 @@ Forks of this project should either:
 - Connect to the same server, that is, wss://server.rplace.tk:443
 - Or use the same app, that is, https://rplace.live
 
-We request forks are not run commercially (That is, it must not generate more than the cost of server upkeep)
+This project is licensed under the GNU LGPL v3, out of goodwill we request forks are
+not run commercially (That is, they should not generate more than the cost of server upkeep).
 
 ### For example,
 

package.json

@@ -1,19 +1,17 @@
 {
   "type": "module",
   "main": "server.js",
-  "license": "GPL-3.0-or-later",
+  "license": "LGPL-3.0-or-later",
   "author": "rplace.live",
   "dependencies": {
     "@datastructures-js/queue": "^4.2.3",
     "basic-repl": "^2.2.1",
     "bun-types": "^1.0.11",
-    "cookie": "^0.5.0",
+    "cookie": "^0.6.0",
     "fs.promises.exists": "^1.1.3",
-    "imagemagick": "^0.1.3",
     "ipapi-sync": "^1.0.0",
     "node-fetch": "^3.2.3",
-    "sha256": "^0.2.0",
-    "skia-canvas": "^1.0.1"
+    "sha256": "^0.2.0"
   },
   "devDependencies": {
     "@typescript-eslint/eslint-plugin": "^6.7.4",
@@ -26,4 +24,4 @@
   "engines": {
     "node": ">=12.11.0"
   }
-}
+}

server-types.ts

@@ -0,0 +1,12 @@
+/** Promise with resolve and reject exposed - Like C# TaskCompletionSource */
+export class PublicPromise<T> {
+    promise: Promise<T>
+    resolve!: (value: T | PromiseLike<T>) => void
+    reject!: (reason?: string) => void
+    constructor() {
+        this.promise = new Promise((resolve, reject) => {
+            this.resolve = resolve
+            this.reject = reject
+        })
+    }
+}