@google-cloud/storage - npm Package Compare versions

Comparing version 6.5.2 to 6.5.3

build/src/acl.js

@@ -18,3 +18,2 @@ "use strict";
 const promisify_1 = require("@google-cloud/promisify");
-const arrify = require("arrify");
 /**
@@ -591,3 +590,3 @@ * Attach functionality to a {@link Storage.acl} instance. This will add an
             if (resp.items) {
-                results = arrify(resp.items).map(this.makeAclObject_);
+                results = resp.items.map(this.makeAclObject_);
             }
@@ -594,0 +593,0 @@ else {

build/src/iam.js

@@ -18,3 +18,2 @@ "use strict";
 const promisify_1 = require("@google-cloud/promisify");
-const arrify = require("arrify");
 const util_1 = require("./util");
@@ -276,3 +275,5 @@ var IAMExceptionMessages;
         const { options, callback: cb } = (0, util_1.normalize)(optionsOrCallback, callback);
-        const permissionsArray = arrify(permissions);
+        const permissionsArray = Array.isArray(permissions)
+            ? permissions
+            : [permissions];
         const req = Object.assign({
@@ -290,3 +291,3 @@ permissions: permissionsArray,
             }
-            const availablePermissions = arrify(resp.permissions);
+            const availablePermissions = resp.permissions;
             const permissionsHash = permissionsArray.reduce((acc, permission) => {
@@ -293,0 +294,0 @@ acc[permission] = availablePermissions.indexOf(permission) > -1;

build/src/nodejs-common/service-object.js

@@ -20,3 +20,2 @@ "use strict";
 const promisify_1 = require("@google-cloud/promisify");
-const arrify = require("arrify");
 const events_1 = require("events");
@@ -256,3 +255,5 @@ const extend = require("extend");
             .join('/');
-        const childInterceptors = arrify(reqOpts.interceptors_);
+        const childInterceptors = Array.isArray(reqOpts.interceptors_)
+            ? reqOpts.interceptors_
+            : [];
         const localInterceptors = [].slice.call(this.interceptors);
@@ -259,0 +260,0 @@ reqOpts.interceptors_ = childInterceptors.concat(localInterceptors);

build/src/nodejs-common/service.js

@@ -19,3 +19,2 @@ "use strict";
  */
-const arrify = require("arrify");
 const extend = require("extend");
@@ -45,3 +44,5 @@ const uuid = require("uuid");
         this.timeout = options.timeout;
-        this.globalInterceptors = arrify(options.interceptors_);
+        this.globalInterceptors = Array.isArray(options.interceptors_)
+            ? options.interceptors_
+            : [];
         this.interceptors = [];
@@ -128,3 +129,6 @@ this.packageJson = config.packageJson;
         const requestInterceptors = this.getRequestInterceptors();
-        arrify(reqOpts.interceptors_).forEach(interceptor => {
+        const interceptorArray = Array.isArray(reqOpts.interceptors_)
+            ? reqOpts.interceptors_
+            : [];
+        interceptorArray.forEach(interceptor => {
             if (typeof interceptor.request === 'function') {
@@ -131,0 +135,0 @@ requestInterceptors.push(interceptor.request);

build/src/storage.js

@@ -20,3 +20,2 @@ "use strict";
 const promisify_1 = require("@google-cloud/promisify");
-const arrify = require("arrify");
 const stream_1 = require("stream");
@@ -428,4 +427,8 @@ const bucket_1 = require("./bucket");
         options = Object.assign({}, options, { apiEndpoint });
-        // Note: EMULATOR_HOST is an experimental configuration variable. Use apiEndpoint instead.
-        const baseUrl = EMULATOR_HOST || `${options.apiEndpoint}/storage/v1`;
+        // Note: EMULATOR_HOST, if present and not overridden, has been applied to
+        // `options` at this point. Also, this uses string concatenation because the
+        // endpoint may contain a base path, and any trailing slash on that will
+        // have been removed, so using the two-arg URL constructor for relative path
+        // resolution won't work.
+        const baseUrl = new URL(options.apiEndpoint + '/storage/v1').href;
         const config = {
@@ -918,3 +921,4 @@ apiEndpoint: options.apiEndpoint,
             }
-            const buckets = arrify(resp.items).map((bucket) => {
+            const itemsArray = resp.items ? resp.items : [];
+            const buckets = itemsArray.map((bucket) => {
                 const bucketInstance = this.bucket(bucket.id);
@@ -943,3 +947,4 @@ bucketInstance.metadata = bucket;
             }
-            const hmacKeys = arrify(resp.items).map((hmacKey) => {
+            const itemsArray = resp.items ? resp.items : [];
+            const hmacKeys = itemsArray.map((hmacKey) => {
                 const hmacKeyInstance = this.hmacKey(hmacKey.accessId, {
@@ -946,0 +951,0 @@ projectId: hmacKey.projectId,

package.json

 {
   "name": "@google-cloud/storage",
   "description": "Cloud Storage Client Library for Node.js",
-  "version": "6.5.2",
+  "version": "6.5.3",
   "license": "Apache-2.0",
@@ -57,3 +57,2 @@ "author": "Google Inc.",
     "abort-controller": "^3.0.0",
-    "arrify": "^2.0.0",
     "async-retry": "^1.3.3",
@@ -60,0 +59,0 @@ "compressible": "^2.0.12",

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

806485

increased by0.14%

Dependency count

17

decreased by-5.56%

Lines of code

16309

increased by0.09%

Dependency changes

• - Removedarrify@^2.0.0