fix: update Docker run command and add hostname validation in reverse…

… proxy middleware
Alokskj · Jan 9, 2025 · a9ec226 · a9ec226
1 parent 1210bce
commit a9ec226
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/.github/workflows/docker.yml b/.github/workflows/docker.yml
@@ -63,4 +63,4 @@ jobs:
                       sudo docker pull alokskj/instahost:latest
                       sudo docker stop instahost
                       sudo docker rm instahost
-                      sudo docker run -d --env-file ./.env -p 5000:80 --name instahost alokskj/instahost
+                      sudo docker run -d --env-file ./.env -p 80:80 --name instahost alokskj/instahost
diff --git a/src/middlewares/reverseProxy.middleware.ts b/src/middlewares/reverseProxy.middleware.ts
@@ -12,9 +12,13 @@ const reverseProxy = async (
     // Extract subdomain from the request hostname
     const hostname = req.hostname; // e.g. 'evara.com' or 'evara.instahost.com' or 'instahost.com'
     console.log(hostname);
+    if (!hostname) {
+        res.status(404).send();
+        return;
+    }
     const host = _config.host; // e.g 'instahost.com'
     let domain;
-    if (hostname.includes(host)) {
+    if (hostname?.includes(host)) {
         if (hostname !== host) {
             domain =
                 hostname.split('.').length > host.split('.').length