Merge pull request #207 from Arquisoft/Despliegue

Despliegue

.github/workflows/pruebaDeploy.yml

@@ -1,7 +1,7 @@
 name: CI for ASW2122
 on:
-  push:
-    branch: [master]
+  release:
+    types: [published]
 
 jobs:
   docker-push-webapp:

restapi/Dockerfile

@@ -2,6 +2,9 @@ FROM node:16.13.1
 LABEL org.opencontainers.image.source https://github.com/Arquisoft/dede_es2a
 COPY . /app
 WORKDIR /app
+
+
 #Install the dependencies
 RUN npm install
+
 CMD [ "npm", "start" ]

restapi/server.ts

@@ -9,16 +9,16 @@ import { pedidoRouter } from "./routes/pedido.router";
 import { usuarioRouter } from "./routes/usuario.router";
 
 const app: Application = express();
-const port: number = 5000;
+const port  = process.env.PORT ||  5000;
 
 
 require('dotenv').config()
 let bd = require('./modules/gestorDB')
 let {config} = require('./modules/cloudinary');
 
 const options: cors.CorsOptions = {
-  origin: ['http://localhost:3000']
-  //origin: ['https://dede-es2a-webapp.herokuapp.com']
+  //origin: ['http://localhost:3000']
+  origin: ['https://dede-es2a-webapp.herokuapp.com']
 };
 
 const metricsMiddleware:RequestHandler = promBundle({includeMethod: true});

webapp/server.ts

@@ -3,7 +3,7 @@ import express,{Application} from 'express';
 //setting the option module to commonjs
 
 var app: Application = express()
-const port: number = 3000;
+const port = process.env.PORT  || 3000;
 
 app.use(express.static('build'))