fiftyone.pipeline.core - npm Package Compare versions

Comparing version 4.4.84 to 4.4.85

evidence.js

@@ -138,3 +138,3 @@ /* *********************************************************************
         'header.protocol',
-        url.parse(request.headers.referer).protocol.replace(':', '')
+        new url.URL(request.headers.referer).protocol.replace(':', '')
       );
@@ -154,3 +154,3 @@ }
 
-    const params = url.parse(request.url, true);
+    const params = new url.URL(request.url, true);
 
@@ -157,0 +157,0 @@ const query = params.query;

examples/examples.test.js

@@ -55,3 +55,3 @@ /* *********************************************************************
 test('pipeline functionality example', () => {
-  testExample({ file: __dirname + '/pipelines.js', portNumber: 3002 });
+  testExample({ file: path.resolve(__dirname, 'pipelines.js'), portNumber: 3002 });
 });
@@ -61,3 +61,3 @@
   testExample(
-    { file: __dirname + '/customFlowElements/simpleEvidenceFlowElement.js', portNumber: 3003 }
+    { file: path.resolve(__dirname, 'customFlowElements', 'simpleEvidenceFlowElement.js'), portNumber: 3003 }
   );
@@ -69,3 +69,3 @@ });
     {
-      file: __dirname + '/customFlowElements/clientSideEvidenceFlowElement.js',
+      file: path.resolve(__dirname, 'customFlowElements', 'clientSideEvidenceFlowElement.js'),
       port: 3004
@@ -72,0 +72,0 @@ }

javascriptbuilder.js

@@ -26,5 +26,6 @@ /* *********************************************************************
 const querystring = require('querystring');
+const path = require('path');
 
 const template = fs.readFileSync(
-  __dirname + '/javascript-templates/JavaScriptResource.mustache',
+  path.resolve(__dirname, 'javascript-templates', 'JavaScriptResource.mustache'),
   'utf8'
@@ -31,0 +32,0 @@ );

package.json

 {
   "name": "fiftyone.pipeline.core",
-  "version": "4.4.84",
+  "version": "4.4.85",
   "description": "Core library for the 51Degrees Pipeline API",
@@ -26,9 +26,2 @@ "keywords": [
     "@types/node": "^15.14.9",
-    "eslint": "^6.8.0",
-    "eslint-config-standard": "^14.1.1",
-    "eslint-plugin-import": "^2.28.1",
-    "eslint-plugin-jsdoc": "^25.4.3",
-    "eslint-plugin-node": "^11.1.0",
-    "eslint-plugin-promise": "^4.3.1",
-    "eslint-plugin-standard": "^4.1.0",
     "jest": "^27.5.1",
@@ -35,0 +28,0 @@ "jest-each": "^27.5.1",

tests/core.test.js

@@ -22,6 +22,7 @@ /* *********************************************************************
  * ********************************************************************* */
+const path = require('path');
 
 const util = require('util');
 const errorMessages = require('../errorMessages');
-const setup = require(__dirname + '/coreTestSetup.js');
+const setup = require(path.resolve(__dirname, 'coreTestSetup.js'));
 const PipelineBuilder = require('../pipelineBuilder');
@@ -221,3 +222,3 @@
 const configPipeline = new PipelineBuilder()
-  .buildFromConfigurationFile(__dirname + '/config.json');
+  .buildFromConfigurationFile(path.resolve(__dirname, 'config.json'));
 const configPipelineFlowData = configPipeline.createFlowData();
@@ -224,0 +225,0 @@

tests/elementData.test.js

@@ -22,17 +22,18 @@ /* *********************************************************************
  * ********************************************************************* */
+const path = require('path');
 
 const util = require('util');
-const errorMessages = require(__dirname +
-  '/../errorMessages');
-const Engine = require(__dirname + '/../../fiftyone.pipeline.engines/engine');
-const PipelineBuilder = require(
-  __dirname + '/../pipelineBuilder'
-);
-const ElementDataDictionary = require(
-  __dirname + '/../elementDataDictionary'
-);
-const BasicListEvidenceKeyFilter = require(
-  __dirname + '/../basicListEvidenceKeyFilter'
-);
 
+const errorMessagesPath = path.resolve(__dirname, '../errorMessages');
+const enginePath = path.resolve(__dirname, '../../fiftyone.pipeline.engines/engine');
+const pipelineBuilderPath = path.resolve(__dirname, '../pipelineBuilder');
+const elementDataDictionaryPath = path.resolve(__dirname, '../elementDataDictionary');
+const basicListEvidenceKeyFilterPath = path.resolve(__dirname, '../basicListEvidenceKeyFilter');
+
+const errorMessages = require(errorMessagesPath);
+const Engine = require(enginePath);
+const PipelineBuilder = require(pipelineBuilderPath);
+const ElementDataDictionary = require(elementDataDictionaryPath);
+const BasicListEvidenceKeyFilter = require(basicListEvidenceKeyFilterPath);
+
 const testEngine = new Engine(
@@ -39,0 +40,0 @@ {

tests/evidence.test.js

@@ -22,4 +22,5 @@ /* *********************************************************************
  * ********************************************************************* */
+const path = require('path');
 
-const setup = require(__dirname + '/coreTestSetup.js');
+const setup = require(path.resolve(__dirname, 'coreTestSetup.js'));
 const PipelineBuilder = require('../pipelineBuilder');
@@ -26,0 +27,0 @@

tests/jsoutput.js

@@ -114,2 +114,3 @@ /* *********************************************************************
     if (callbackCounter === 0) {
+      resolve();
     } else if (callbackCounter < 0) {
@@ -116,0 +117,0 @@ reject('Too many callbacks.');

tests/logs-errors-and-stops.test.js

@@ -22,7 +22,8 @@ /* *********************************************************************
  * ********************************************************************* */
+const path = require('path');
 
 const util = require('util');
-const errorMessages = require('../errorMessages');
-const setup = require(__dirname + '/coreTestSetup.js');
-const PipelineBuilder = require('../pipelineBuilder');
+const errorMessages = require(path.resolve(__dirname, '..', 'errorMessages'));
+const setup = require(path.resolve(__dirname, 'coreTestSetup.js'));
+const PipelineBuilder = require(path.resolve(__dirname, '..', 'pipelineBuilder'));
 
@@ -29,0 +30,0 @@ const syncPipeline = new PipelineBuilder()

tests/setHeadersElement.test.js

@@ -22,4 +22,5 @@ /* *********************************************************************
  * ********************************************************************* */
+const path = require('path');
 
-const setup = require(__dirname + '/coreTestSetup.js');
+const setup = require(path.resolve(__dirname, 'coreTestSetup.js'));
 const PipelineBuilder = require('../pipelineBuilder');
@@ -26,0 +27,0 @@ const Helpers = require('../helpers');

New alerts

Dynamic require

Supply chain risk

Dynamic require can indicate the package is performing dangerous or unsafe dynamic code execution.

Found 1 instance in 1 package

Fixed alerts

Dynamic require

Supply chain risk

Dynamic require can indicate the package is performing dangerous or unsafe dynamic code execution.

Found 1 instance in 1 package

Improved metrics

Dev dependency count

4

decreased by-63.64%

Lines of code

4636

increased by0.11%

Worsened metrics

Total package byte prevSize

222872

decreased by-0.48%

Number of package files

59

decreased by-1.67%

Number of low supply chain risk alerts

23

increased by9.52%

No dependency changes