[SDKS-2199] Add Cors. Fix listener destroy. Update changes file

CHANGES.txt

@@ -1,6 +1,14 @@
+2.1.1 (XXX XX, 2022)
+  - Updated SDK version to v10.22.0 which is the latest to the date.
+  - Added new env variable to enable cors
+  - Added Multi environment support
+  - Added new env variables:
+    - SPLIT_EVALUATOR_ENVIRONMENTS
+    - SPLIT_EVALUATOR_ENABLE_CORS
+
 2.1.0 (Jul 15, 2022)
  - Added support for attribute values to be sent as JSON in a POST version of the get treatment endpoints, to avoid query param limitations.
- - Updated the SDK version to 10.20.0 which is the latest to the date. 
+ - Updated the SDK version to 10.20.0 which is the latest to the date.
  - Updated npm dependencies for vulnerability fixes.
  - Updated base image to node:16.16.0-alpine3.16
 

__tests__/cors.test.js

@@ -0,0 +1,22 @@
+const request = require('supertest');
+
+describe('CORS ', () => {
+  beforeEach(() => {
+    jest.resetModules();
+  });
+
+  test('', async () => {
+    process.env.SPLIT_EVALUATOR_ENABLE_CORS = true;
+    const app = require('../app');
+    const { headers } = await request(app).get('/');
+    expect(headers['access-control-allow-origin']).toEqual('*');
+  });
+
+  test('default', async () => {
+    delete process.env.SPLIT_EVALUATOR_ENABLE_CORS;
+    const app = require('../app');
+    const { headers } = await request(app).get('/');
+    expect(headers['access-control-allow-origin']).toEqual(undefined);
+  });
+
+});

app.js

@@ -17,6 +17,12 @@ const adminRouter = require('./admin/admin.router');
 
 // Utils
 const utils = require('./utils/utils');
+// Cors
+if (process.env.SPLIT_EVALUATOR_ENABLE_CORS === 'true') {
+  console[console.warn ? 'warn' : 'log']('Cors enabled');
+  const cors = require('cors');
+  app.use(cors());
+}
 
 app.use(morgan('tiny'));
 

listener/manager.js

@@ -74,6 +74,7 @@ const ImpressionManagerFactory = (function(){
       return instance;
     },
     async destroy() {
+      if (!instance) return;
       await instance.destroy();
       instance = undefined;
     },

package.json

@@ -41,6 +41,7 @@
   "dependencies": {
     "@splitsoftware/splitio": "10.22.0",
     "config": "1.25.1",
+    "cors": "^2.8.5",
     "express": "^4.17.1",
     "morgan": "^1.9.1",
     "npm": "^8.13.2",