Merge pull request #18 from acemilyalcin/fix-dockerfile

Fixed Dockerfile and docker-compose.yml

Author: acemilyalcin

.dockerignore

@@ -0,0 +1,7 @@
+node_modules
+npm-debug.log
+Dockerfile
+.dockerignore
+.git
+.gitignore
+docker-compose.yml

Dockerfile

@@ -0,0 +1,19 @@
+FROM node:24-alpine
+
+# Create app user and group
+RUN addgroup app && adduser -S -G app app
+
+WORKDIR /usr/src/app
+
+# Copy package files and install dependencies as root
+COPY package*.json ./
+RUN npm ci --omit=dev
+
+# Copy rest of the app
+COPY . .
+
+# Switch to non-root user for runtime
+USER app
+
+EXPOSE 3000
+CMD ["node", "app.js"]

app.js

@@ -9,7 +9,7 @@ var express = require('express');
 
 var app = express();
 
-app.set('port', process.env.PORT || 3005); // GİRİŞ PORTU AYARLANDI
+app.set('port', process.env.PORT || 3000); // GİRİŞ PORTU AYARLANDI
 app.set('views', __dirname + '/app/server/views'); // VIEW KLASÖRÜ TANITILDI
 app.set('view engine', 'ejs'); // VIEW ENGINE AYARLANDI
 app.use(express.static(__dirname + '/app/public')); // KULLANICILAR TARAFINDAN ERİŞİLEBİLEN KLASÖR TANIMLANDI

docker-compose.yml

@@ -1,14 +1,8 @@
-version: '3.7'
-
 services:
-  environment:
-    container_name: environment
+  app:
+    container_name: app
     build:
       context: .
-      dockerfile: dockerfile
+      dockerfile: Dockerfile
     ports:
-      - 3005:3005
-    stdin_open: true
-    environment:
-      - CHOKIDAR_USEPOLLING=true
-    command: node app.js
+      - "3000:3000"

dockerfile

@@ -4,9 +4,7 @@
   "description": "Sample Node.js Project",
   "main": "app.js",
   "scripts": {
-    "start": "node app.js",
-    "test": "echo \"Error: no test specified\" && exit 1",
-    "build": "echo 'Building the project...' && exit 0" // Replace this with your actual build command
+    "start": "node app.js"
   },
   "author": "Adem Cemil YALCIN",
   "license": "ISC",