1.1.15

.gitlab-ci.yml

@@ -26,6 +26,7 @@ mirror:
 snyk:
   stage: security
   script:
+    - npmci npm prepare
     - npmci command npm install -g snyk
     - npmci command npm install --ignore-scripts
     - npmci command snyk test
@@ -36,21 +37,11 @@ snyk:
 # ====================
 # test stage
 # ====================
-testLEGACY:
-  stage: test
-  script:
-  - npmci node install legacy
-  - npmci npm install
-  - npmci npm test
-  coverage: /\d+.?\d+?\%\s*coverage/
-  tags:
-  - docker
-  - notpriv
-  allow_failure: true
 
 testLTS:
   stage: test
   script:
+  - npmci npm prepare
   - npmci node install lts
   - npmci npm install
   - npmci npm test
@@ -62,6 +53,7 @@ testLTS:
 testSTABLE:
   stage: test
   script:
+  - npmci npm prepare
   - npmci node install stable
   - npmci npm install
   - npmci npm test
@@ -118,6 +110,7 @@ pages:
   stage: metadata
   script:
     - npmci command npm install -g typedoc typescript
+    - npmci npm prepare
     - npmci npm install
     - npmci command typedoc --module "commonjs" --target "ES2016" --out public/ ts/
   tags:
@@ -130,13 +123,3 @@ pages:
     paths:
     - public
   allow_failure: true
-
-windowsCompatibility:
-  image: stefanscherer/node-windows:10-build-tools
-  stage: metadata
-  script:
-  - npm install & npm test
-  coverage: /\d+.?\d+?\%\s*coverage/
-  tags:
-  - windows
-  allow_failure: true

README.md

@@ -1,25 +1,20 @@
-# smartrequest
+# @pushrocks/smartrequest
-
 dropin replacement for request
 
-## Availabililty
+## Availabililty and Links
-
+* [npmjs.org (npm package)](https://www.npmjs.com/package/@pushrocks/smartrequest)
-[![npm](https://push.rocks/assets/repo-button-npm.svg)](https://www.npmjs.com/package/smartrequest)
+* [gitlab.com (source)](https://gitlab.com/pushrocks/smartrequest)
-[![git](https://push.rocks/assets/repo-button-git.svg)](https://GitLab.com/pushrocks/smartrequest)
+* [github.com (source mirror)](https://github.com/pushrocks/smartrequest)
-[![git](https://push.rocks/assets/repo-button-mirror.svg)](https://github.com/pushrocks/smartrequest)
+* [docs (typedoc)](https://pushrocks.gitlab.io/smartrequest/)
-[![docs](https://push.rocks/assets/repo-button-docs.svg)](https://pushrocks.gitlab.io/smartrequest/)
 
 ## Status for master
-
+[![build status](https://gitlab.com/pushrocks/smartrequest/badges/master/build.svg)](https://gitlab.com/pushrocks/smartrequest/commits/master)
-[![build status](https://GitLab.com/pushrocks/smartrequest/badges/master/build.svg)](https://GitLab.com/pushrocks/smartrequest/commits/master)
+[![coverage report](https://gitlab.com/pushrocks/smartrequest/badges/master/coverage.svg)](https://gitlab.com/pushrocks/smartrequest/commits/master)
-[![coverage report](https://GitLab.com/pushrocks/smartrequest/badges/master/coverage.svg)](https://GitLab.com/pushrocks/smartrequest/commits/master)
+[![npm downloads per month](https://img.shields.io/npm/dm/@pushrocks/smartrequest.svg)](https://www.npmjs.com/package/@pushrocks/smartrequest)
-[![npm downloads per month](https://img.shields.io/npm/dm/smartrequest.svg)](https://www.npmjs.com/package/smartrequest)
+[![Known Vulnerabilities](https://snyk.io/test/npm/@pushrocks/smartrequest/badge.svg)](https://snyk.io/test/npm/@pushrocks/smartrequest)
-[![Dependency Status](https://david-dm.org/pushrocks/smartrequest.svg)](https://david-dm.org/pushrocks/smartrequest)
+[![TypeScript](https://img.shields.io/badge/TypeScript->=%203.x-blue.svg)](https://nodejs.org/dist/latest-v10.x/docs/api/)
-[![bitHound Dependencies](https://www.bithound.io/github/pushrocks/smartrequest/badges/dependencies.svg)](https://www.bithound.io/github/pushrocks/smartrequest/master/dependencies/npm)
+[![node](https://img.shields.io/badge/node->=%2010.x.x-blue.svg)](https://nodejs.org/dist/latest-v10.x/docs/api/)
-[![bitHound Code](https://www.bithound.io/github/pushrocks/smartrequest/badges/code.svg)](https://www.bithound.io/github/pushrocks/smartrequest)
+[![JavaScript Style Guide](https://img.shields.io/badge/code%20style-prettier-ff69b4.svg)](https://prettier.io/)
-[![TypeScript](https://img.shields.io/badge/TypeScript-2.x-blue.svg)](https://nodejs.org/dist/latest-v6.x/docs/api/)
-[![node](https://img.shields.io/badge/node->=%206.x.x-blue.svg)](https://nodejs.org/dist/latest-v6.x/docs/api/)
-[![JavaScript Style Guide](https://img.shields.io/badge/code%20style-standard-brightgreen.svg)](http://standardjs.com/)
 
 ## Usage
 
@@ -66,4 +61,9 @@ smartrequest.get('https://example.com/bigfile.mp4', optionsArg, true).then(res =
 })
 ```
 
-[![npm](https://push.rocks/assets/repo-header.svg)](https://push.rocks)
+For further information read the linked docs at the top of this readme.
+
+> MIT licensed | **©** [Lossless GmbH](https://lossless.gmbh)
+| By using this npm module you agree to our [privacy policy](https://lossless.gmbH/privacy.html)
+
+[![repo-footer](https://pushrocks.gitlab.io/assets/repo-footer.svg)](https://maintainedby.lossless.com)

npmextra.json

@@ -3,9 +3,17 @@
     "coverageTreshold": 50
   },
   "npmci": {
-    "npmGlobalTools": [
+    "npmGlobalTools": [],
-      "npmts"
-    ],
     "npmAccessLevel": "public"
+  },
+  "gitzone": {
+    "module": {
+      "githost": "gitlab.com",
+      "gitscope": "pushrocks",
+      "gitrepo": "smartrequest",
+      "shortDescription": "dropin replacement for request",
+      "npmPackagename": "@pushrocks/smartrequest",
+      "license": "MIT"
+    }
   }
 }

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pushrocks/smartrequest",
-  "version": "1.1.13",
+  "version": "1.1.15",
   "private": false,
   "description": "dropin replacement for request",
   "main": "dist/index.js",
@@ -23,15 +23,15 @@
   },
   "homepage": "https://gitlab.com/pushrocks/smartrequest#README",
   "dependencies": {
-    "@pushrocks/smartpromise": "^2.0.5",
+    "@pushrocks/smartpromise": "^3.0.2",
     "@types/form-data": "^2.2.1",
-    "form-data": "^2.3.2"
+    "form-data": "^2.3.3"
   },
   "devDependencies": {
-    "@gitzone/tsbuild": "^2.0.22",
+    "@gitzone/tsbuild": "^2.1.8",
-    "@gitzone/tsrun": "^1.1.12",
+    "@gitzone/tsrun": "^1.2.5",
-    "@gitzone/tstest": "^1.0.15",
+    "@gitzone/tstest": "^1.0.20",
-    "@pushrocks/tapbundle": "^3.0.5",
+    "@pushrocks/tapbundle": "^3.0.9",
-    "@types/node": "^10.5.8"
+    "@types/node": "^11.13.6"
   }
 }

test/test.ts

@@ -23,17 +23,18 @@ tap.test('should post a JSON document over http', async () => {
 });
 
 tap.skip.test('should deal with unix socks', async () => {
-  const socketResponse = await smartrequest.request('http://unix:/var/run/docker.sock:/containers/json', {
+  const socketResponse = await smartrequest.request(
+    'http://unix:/var/run/docker.sock:/containers/json',
+    {
       headers: {
-      "Content-Type": "application/json",
+        'Content-Type': 'application/json',
-      "Host": "docker.sock"
+        Host: 'docker.sock'
       }
-  });
+    }
+  );
   console.log(socketResponse.body);
 });
 
-tap.skip.test('should correctly upload a file using formData', async () => {
+tap.skip.test('should correctly upload a file using formData', async () => {});
-  
-})
 
 tap.start();

ts/smartrequest.formdata.ts

@@ -1,25 +1,20 @@
-import * as plugins from "./smartrequest.plugins";
+import * as plugins from './smartrequest.plugins';
-import * as interfaces from "./smartrequest.interfaces";
+import * as interfaces from './smartrequest.interfaces';
-import { request } from "./smartrequest.request";
+import { request } from './smartrequest.request';
 
 /**
  * the interfae for FormFieldData
  */
 export interface IFormField {
   name: string;
-  type: "string" | "filePath" | "Buffer";
+  type: 'string' | 'filePath' | 'Buffer';
   payload: string;
 }
 
-const appendFormField = async (
+const appendFormField = async (formDataArg: plugins.formData, formDataField: IFormField) => {
-  formDataArg: plugins.formData,
+  if (formDataField.type === 'filePath') {
-  formDataField: IFormField
+    let fileData = plugins.fs.readFileSync(plugins.path.join(process.cwd(), formDataField.payload));
-) => {
+    formDataArg.append('file', fileData, {
-  if (formDataField.type === "filePath") {
-    let fileData = plugins.fs.readFileSync(
-      plugins.path.join(process.cwd(), formDataField.payload)
-    );
-    formDataArg.append("file", fileData, {
       filename: 'upload.pdf',
       contentType: 'application/pdf'
     });
@@ -38,7 +33,7 @@ export const postFormData = async (
   const requestOptions = Object.assign({}, optionsArg, {
     method: 'POST',
     headers: {
-      ...(optionsArg.headers),
+      ...optionsArg.headers,
       ...form.getHeaders()
     },
     requestBody: form

ts/smartrequest.request.ts

@@ -1,8 +1,8 @@
-import * as https from "https";
+import * as https from 'https';
-import * as plugins from "./smartrequest.plugins";
+import * as plugins from './smartrequest.plugins';
-import * as interfaces from "./smartrequest.interfaces";
+import * as interfaces from './smartrequest.interfaces';
 
-import { IncomingMessage } from "http";
+import { IncomingMessage } from 'http';
 
 export interface IExtendedIncomingMessage extends IncomingMessage {
   body: any;
@@ -13,12 +13,12 @@ const buildUtf8Response = (
 ): Promise<IExtendedIncomingMessage> => {
   let done = plugins.smartpromise.defer<IExtendedIncomingMessage>();
   // Continuously update stream with data
-  let body = "";
+  let body = '';
-  incomingMessageArg.on("data", function(chunkArg) {
+  incomingMessageArg.on('data', function(chunkArg) {
     body += chunkArg;
   });
 
-  incomingMessageArg.on("end", function() {
+  incomingMessageArg.on('end', function() {
     try {
       (incomingMessageArg as IExtendedIncomingMessage).body = JSON.parse(body);
     } catch (err) {
@@ -47,8 +47,8 @@ const parseSocketPathAndRoute = (stringToParseArg: string) => {
   return {
     socketPath: result[1],
     path: result[2]
-  }
+  };
-}
+};
 
 export let request = async (
   domainArg: string,
@@ -68,30 +68,29 @@ export let request = async (
 
   // determine if unixsock
   if (testForUnixSock(domainArg)) {
-    const detailedUnixPath = parseSocketPathAndRoute(optionsArg.path)
+    const detailedUnixPath = parseSocketPathAndRoute(optionsArg.path);
     optionsArg.socketPath = detailedUnixPath.socketPath;
     optionsArg.path = detailedUnixPath.path;
   }
 
   // lets determine the request module to use
   const requestModule = (() => {
-    if (parsedUrl.protocol === "https:") {
+    if (parsedUrl.protocol === 'https:') {
       return plugins.https;
-    } else if (parsedUrl.protocol === "http:") {
+    } else if (parsedUrl.protocol === 'http:') {
       return plugins.http;
     } else {
       throw new Error(`unsupported protocol: ${parsedUrl.protocol}`);
     }
   })() as typeof plugins.https;
 
-
   // lets perform the actual request
   let request = requestModule.request(optionsArg);
 
   // lets write the requestBody
   if (optionsArg.requestBody) {
     if (!(optionsArg.requestBody instanceof plugins.formData)) {
-      if(typeof optionsArg.requestBody !== "string") {
+      if (typeof optionsArg.requestBody !== 'string') {
         optionsArg.requestBody = JSON.stringify(optionsArg.requestBody);
       }
       request.write(optionsArg.requestBody);
@@ -106,16 +105,16 @@ export let request = async (
   }
 
   // lets handle an error
-  request.on("error", e => {
+  request.on('error', e => {
     console.error(e);
   });
 
   // lets handle the response
-  request.on("response", async response => {
+  request.on('response', async response => {
     if (streamArg) {
       done.resolve(response);
     } else {
-      const builtResponse = await buildUtf8Response(response)
+      const builtResponse = await buildUtf8Response(response);
       done.resolve(builtResponse);
     }
   });

tslint.json

@@ -1,3 +1,17 @@
 {
-    "extends": "tslint-config-standard"
+  "extends": ["tslint:latest", "tslint-config-prettier"],
+  "rules": {
+    "semicolon": [true, "always"],
+    "no-console": false,
+    "ordered-imports": false,
+    "object-literal-sort-keys": false,
+    "member-ordering": {
+      "options":{
+        "order": [
+          "static-method"
+        ]
+      }
+    }
+  },
+  "defaultSeverity": "warning"
 }