remove cors verify in openshift

bcgov · Nov 28, 2024 · 3a9ee87 · 3a9ee87
1 parent 721022a
commit 3a9ee87
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 8 deletions.
diff --git a/app/src/forms/bcgeoaddress/routes.js b/app/src/forms/bcgeoaddress/routes.js
@@ -1,12 +1,19 @@
+const cors = require('cors');
 const routes = require('express').Router();
+const { Development } = require('../common/constants');
 
 const controller = require('./controller');
 
-routes.get('/address', async (req, res, next) => {
+// need to allow cors for OPTIONS call (localhost only)
+// formio component will call OPTIONS pre-flight
+routes.options('/address', cors({ origin: Development.LOCALHOST_ORIGIN }));
+routes.options('/advance/address', cors({ origin: Development.LOCALHOST_ORIGIN }));
+
+routes.get('/address', cors({ origin: Development.LOCALHOST_ORIGIN }), async (req, res, next) => {
   await controller.searchBCGeoAddress(req, res, next);
 });
 
-routes.get('/advance/address', async (req, res, next) => {
+routes.get('/advance/address', cors({ origin: Development.LOCALHOST_ORIGIN }), async (req, res, next) => {
   await controller.advanceSearchBCGeoAddress(req, res, next);
 });
 

diff --git a/app/src/forms/proxy/routes.js b/app/src/forms/proxy/routes.js
@@ -1,15 +1,10 @@
-const cors = require('cors');
 const routes = require('express').Router();
 
 const { currentUser } = require('../auth/middleware/userAccess');
 const controller = require('./controller');
 
-// need to allow cors for OPTIONS call
-// formio component will call OPTIONS pre-flight
-routes.options('/external', cors());
-
 // called with encrypted headers, no current user!!!
-routes.get('/external', cors(), async (req, res, next) => {
+routes.get('/external', async (req, res, next) => {
   await controller.callExternalApi(req, res, next);
 });