reqlib - npm Package Compare versions

Comparing version 1.0.6 to 1.0.7

dist/index.js

@@ -246,2 +246,3 @@ "use strict";
       var executeRequest,
+          requestContentType,
           self = this,
@@ -273,19 +274,23 @@ state = {};
 
-      if (!headerExists(options.headers, HTTP_HEADERS.CONTENT_TYPE)) {
+      requestContentType = coalesce(options.headers[HTTP_HEADERS.CONTENT_TYPE], options.headers[HTTP_HEADERS.CONTENT_TYPE.toLowerCase()]); // default the content type if not provided...
+
+      if (!requestContentType) {
         // apply application/json header as default (this is opinionated)
-        options.headers[HTTP_HEADERS.CONTENT_TYPE] = 'application/json'; // serialize data (if provided) as JSON
+        options.headers[HTTP_HEADERS.CONTENT_TYPE] = 'application/json';
+        requestContentType = 'application/json';
+      } // ensure serialization of data
 
-        if (isObject(state.data)) {
-          state.data = (0, _stringify.default)(state.data);
-        }
-      } // ensure state data is
 
-
-      if (isObject(state.data) && state.data.toString) {
-        state.data = state.data.toString();
+      if (RE_CONTENT_TYPE_JSON.test(requestContentType)) {
+        state.data = (0, _stringify.default)(data);
+      } else if (data && data.toString && typeof data.toString === 'function') {
+        state.data = data.toString();
       } // apply content length header
 
 
-      options.headers[HTTP_HEADERS.CONTENT_LENGTH] = options.headers[HTTP_HEADERS.CONTENT_LENGTH] || Buffer.byteLength(state.data); // setup failover if applicable
+      if (typeof state.data === 'string') {
+        options.headers[HTTP_HEADERS.CONTENT_LENGTH] = options.headers[HTTP_HEADERS.CONTENT_LENGTH] || Buffer.byteLength(state.data);
+      } // setup failover if applicable
 
+
       ['host', 'hostname', 'hostnames', 'hosts'].forEach(function (field) {
@@ -510,3 +515,3 @@ var key = RE_ENDS_WITH_S.test(field) ? field.slice(0, -1) : field; // if the host or hostname field value is an Array
 
-          if (state.data) {
+          if (state.data && (typeof state.data === 'string' || Buffer.isBuffer(state.data))) {
             client.write(state.data);
@@ -513,0 +518,0 @@ } // finish up the client stream and end to send

history.md

@@ -0,1 +1,5 @@
+# v1.0.7 - 03/12/2019
+
+* Adjusted how content-length header is calculated and applied for JSON data
+
 # v1.0.6 - 03/05/2019
@@ -2,0 +6,0 @@

package.json

 {
   "name": "reqlib",
-  "version": "1.0.6",
+  "version": "1.0.7",
   "description": "A library to simplify REST API requests and to provide utilities for handling common REST API consumer scenarios ",
@@ -5,0 +5,0 @@ "main": "dist",

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

83059

increased by1.39%

Lines of code

802

increased by0.5%

No dependency changes