Merge pull request #586 from pennlabs/revert-583-bump-node

Revert "Bump node to 18"
pennlabs · Feb 10, 2024 · a657e2b · a657e2b
2 parents 1cfbae0 + 8039f4f
commit a657e2b
Show file tree

Hide file tree

Showing 7 changed files with 11 additions and 12 deletions.
diff --git a/.github/workflows/shared-build-deploy.yaml b/.github/workflows/shared-build-deploy.yaml
@@ -55,7 +55,6 @@ jobs:
         product: [review, plan, alert]
     with:
       path: frontend/${{ matrix.product }}
-      nodeVersion: 18.19.0
 
   publish-frontend:
     uses: pennlabs/shared-actions/.github/workflows/[email protected]

diff --git a/frontend/alert/Dockerfile b/frontend/alert/Dockerfile
@@ -1,4 +1,4 @@
-FROM node:18-bookworm
+FROM node:10-buster
 
 LABEL maintainer="Penn Labs"
 

diff --git a/frontend/alert/package.json b/frontend/alert/package.json
@@ -33,9 +33,9 @@
     "typescript": "^3.9.5"
   },
   "scripts": {
-    "dev": "NODE_OPTIONS=--openssl-legacy-provider node server.js",
-    "build": "NODE_OPTIONS=--openssl-legacy-provider next build",
-    "start": "NODE_OPTIONS=--openssl-legacy-provider NODE_ENV=production node server.js",
+    "dev": "node server.js",
+    "build": "next build",
+    "start": "NODE_ENV=production node server.js",
     "test": "echo no tests yet",
     "codecov": "echo no codecov",
     "lint": "eslint .",

diff --git a/frontend/plan/Dockerfile b/frontend/plan/Dockerfile
@@ -1,4 +1,4 @@
-FROM node:18-bookworm
+FROM node:10-buster
 
 LABEL maintainer="Penn Labs"
 

diff --git a/frontend/plan/package.json b/frontend/plan/package.json
@@ -4,8 +4,8 @@
   "private": true,
   "scripts": {
     "dev": "export NODE_OPTIONS=--openssl-legacy-provider && node server.js",
-    "build": "NODE_OPTIONS=--openssl-legacy-provider next build",
-    "start": "NODE_OPTIONS=--openssl-legacy-provider NODE_ENV=production node server.js",
+    "build": "next build",
+    "start": "NODE_ENV=production node server.js",
     "test": "echo no tests yet",
     "codecov": "echo no codecov",
     "lint": "eslint . && tsc --noemit",

diff --git a/frontend/review/Dockerfile b/frontend/review/Dockerfile
@@ -1,4 +1,4 @@
-FROM node:18-bookworm as build-deps
+FROM node:10-buster as build-deps
 
 WORKDIR /app/
 

diff --git a/frontend/review/package.json b/frontend/review/package.json
@@ -23,9 +23,9 @@
     "workerize-loader": "^1.3.0"
   },
   "scripts": {
-    "dev": "NODE_OPTIONS=--openssl-legacy-provider react-scripts start",
-    "start": "NODE_OPTIONS=--openssl-legacy-provider react-scripts start",
-    "build": "NODE_OPTIONS=--openssl-legacy-provider react-scripts build",
+    "dev": "react-scripts start",
+    "start": "react-scripts start",
+    "build": "react-scripts build",
     "test": "exit 0",
     "eject": "react-scripts eject",
     "lint": "prettier --check \"src/**/*.js\" && eslint \"./src/**/*.js\"",