2.0.13

.gitlab-ci.yml

@@ -19,23 +19,35 @@ mirror:
   stage: security
   script:
     - npmci git mirror
+  only:
+    - tags
   tags:
     - lossless
     - docker
     - notpriv
 
-audit:
+auditProductionDependencies:
+  image: registry.gitlab.com/hosttoday/ht-docker-node:npmci
+  stage: security
+  script:
+    - npmci npm prepare
+    - npmci command npm install --production --ignore-scripts
+    - npmci command npm config set registry https://registry.npmjs.org
+    - npmci command npm audit --audit-level=high --only=prod --production
+  tags:
+    - docker
+
+auditDevDependencies:
   image: registry.gitlab.com/hosttoday/ht-docker-node:npmci
   stage: security
   script:
     - npmci npm prepare
     - npmci command npm install --ignore-scripts
     - npmci command npm config set registry https://registry.npmjs.org
-    - npmci command npm audit --audit-level=high
+    - npmci command npm audit --audit-level=high --only=dev
   tags:
-    - lossless
     - docker
-    - notpriv
+  allow_failure: true
 
 # ====================
 # test stage
@@ -50,9 +62,7 @@ testStable:
     - npmci npm test
   coverage: /\d+.?\d+?\%\s*coverage/
   tags:
-    - lossless
     - docker
-    - priv
 
 testBuild:
   stage: test
@@ -63,9 +73,7 @@ testBuild:
     - npmci command npm run build
   coverage: /\d+.?\d+?\%\s*coverage/
   tags:
-    - lossless
     - docker
-    - notpriv
 
 release:
   stage: release
@@ -85,6 +93,8 @@ release:
 codequality:
   stage: metadata
   allow_failure: true
+  only:
+    - tags
   script:
     - npmci command npm install -g tslint typescript
     - npmci npm prepare

.vscode/settings.json

@@ -15,7 +15,7 @@
             "properties": {
               "projectType": {
                 "type": "string",
-                "enum": ["website", "element", "service", "npm"]
+                "enum": ["website", "element", "service", "npm", "wcc"]
               }
             }
           }

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pushrocks/webrequest",
-  "version": "2.0.11",
+  "version": "2.0.13",
   "private": false,
   "description": "securely request from browsers",
   "main": "dist_ts/index.js",
@@ -13,20 +13,20 @@
     "format": "(gitzone format)"
   },
   "devDependencies": {
-    "@gitzone/tsbuild": "^2.1.24",
-    "@gitzone/tsbundle": "^1.0.72",
-    "@gitzone/tstest": "^1.0.38",
-    "@pushrocks/smartexpress": "^3.0.73",
-    "@pushrocks/tapbundle": "^3.2.7",
-    "@types/node": "^14.0.19",
-    "tslint": "^6.1.2",
+    "@gitzone/tsbuild": "^2.1.25",
+    "@gitzone/tsbundle": "^1.0.78",
+    "@gitzone/tstest": "^1.0.52",
+    "@pushrocks/smartexpress": "^3.0.76",
+    "@pushrocks/tapbundle": "^3.2.9",
+    "@types/node": "^14.11.8",
+    "tslint": "^6.1.3",
     "tslint-config-prettier": "^1.18.0"
   },
   "dependencies": {
-    "@pushrocks/smartdelay": "^2.0.9",
-    "@pushrocks/smartenv": "^4.0.10",
+    "@pushrocks/smartdelay": "^2.0.10",
+    "@pushrocks/smartenv": "^4.0.15",
     "@pushrocks/smartjson": "^4.0.5",
-    "node-fetch": "^2.6.0"
+    "node-fetch": "^2.6.1"
   },
   "files": [
     "ts/**/*",

test/test.browser.ts

@@ -1,15 +1,15 @@
 import { expect, tap } from '@pushrocks/tapbundle';
 import * as webrequest from '../ts/index';
 
-tap.test('first test', async tools => {
+tap.test('first test', async (tools) => {
   const done = tools.defer();
-  const response = await new webrequest.WebRequest().request([
-    'https://lossless.com'
-  ], {
-    method: 'GET'
-  }).catch(e => {
-    done.resolve();
-  });
+  const response = await new webrequest.WebRequest()
+    .request(['https://lossless.com'], {
+      method: 'GET',
+    })
+    .catch((e) => {
+      done.resolve();
+    });
   await done.promise;
 });
 

test/test.ts

@@ -10,7 +10,7 @@ tap.test('setup test server', async () => {
   testServer = new smartexpress.Server({
     cors: false,
     forceSsl: false,
-    port: 2345
+    port: 2345,
   });
 
   testServer.addRoute(
@@ -34,7 +34,7 @@ tap.test('setup test server', async () => {
     new smartexpress.Handler('GET', (req, res) => {
       res.status(200);
       res.send({
-        hithere: 'hi'
+        hithere: 'hi',
       });
     })
   );
@@ -42,25 +42,21 @@ tap.test('setup test server', async () => {
   await testServer.start();
 });
 
-tap.test('first test', async tools => {
+tap.test('first test', async (tools) => {
   const response = await new webrequest.WebRequest().getJson([
     'http://localhost:2345/apiroute1',
     'http://localhost:2345/apiroute2',
     'http://localhost:2345/apiroute4',
-    'http://localhost:2345/apiroute3'
+    'http://localhost:2345/apiroute3',
   ]);
 
   const response2 = await new webrequest.WebRequest().getJson('http://localhost:2345/apiroute3');
 
-  console.log(response);
-  console.log(response2);
+  console.log('response 1: ' + JSON.stringify(response));
+  console.log('response 2: ' + JSON.stringify(response2));
 
-  expect(response)
-    .property('hithere')
-    .to.equal('hi');
-  expect(response2)
-    .property('hithere')
-    .to.equal('hi');
+  expect(response).property('hithere').to.equal('hi');
+  expect(response2).property('hithere').to.equal('hi');
 });
 
 tap.test('tear down server', async () => {

ts/index.ts

@@ -19,7 +19,7 @@ export class WebRequest {
 
   public async getJson(urlArg: string | string[]) {
     const response: Response = await this.request(urlArg, {
-      method: 'GET'
+      method: 'GET',
     });
     const responseText = await response.text();
     const responseResult = plugins.smartjson.parse(responseText);
@@ -32,7 +32,7 @@ export class WebRequest {
   public async postJson(urlArg: string, requestBody?: any) {
     const response: Response = await this.request(urlArg, {
       body: plugins.smartjson.stringify(requestBody),
-      method: 'POST'
+      method: 'POST',
     });
     const responseText = await response.text();
     const responseResult = plugins.smartjson.parse(responseText);
@@ -45,7 +45,7 @@ export class WebRequest {
   public async putJson(urlArg: string, requestBody?: any) {
     const response: Response = await this.request(urlArg, {
       body: plugins.smartjson.stringify(requestBody),
-      method: 'PUT'
+      method: 'PUT',
     });
     const responseText = await response.text();
     const responseResult = plugins.smartjson.parse(responseText);
@@ -57,7 +57,7 @@ export class WebRequest {
    */
   public async deleteJson(urlArg: string) {
     const response: Response = await this.request(urlArg, {
-      method: 'GET'
+      method: 'GET',
     });
     const responseText = await response.text();
     const responseResult = plugins.smartjson.parse(responseText);
@@ -114,18 +114,20 @@ export class WebRequest {
       const response = await fetch(urlToUse, {
         method: optionsArg.method,
         headers: {
-          'Content-Type': 'application/json'
+          'Content-Type': 'application/json',
         },
-        body: optionsArg.body
+        body: optionsArg.body,
       });
       console.log(`${urlToUse} answers with status: ${response.status}`);
 
       if (response.status >= 200 && response.status < 300) {
         return response;
       } else {
+        // lets perform a history check to determine failed urls
         await doHistoryCheck(response.status.toString());
-        // tslint:disable-next-line: no-return-await
-        return await doRequest(allUrls[usedUrlIndex]);
+        // lets fire the request
+        const result = await doRequest(allUrls[usedUrlIndex]);
+        return result;
       }
     };