remove shell-escaping url in S3ZipContent (#200)

serverless · Mar 6, 2020 · 44b5217 · 44b5217
1 parent b037631
commit 44b5217
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 14 deletions.
diff --git a/.travis.yml b/.travis.yml
@@ -9,8 +9,8 @@ services:
 - docker
 env:
   global:
-  - KUBELESS_VERSION: v1.0.0-alpha.8
-  - KUBELESS_KAFKA_VERSION: v1.0.0-beta.0
+  - KUBELESS_VERSION: v1.0.6
+  - KUBELESS_KAFKA_VERSION: v1.0.2
   - MINIKUBE_VERSION: v0.25.2
   - REPO_DOMAIN: serverless
   - REPO_NAME: serverless-kubeless

diff --git a/lib/strategy/s3_zip_content.js b/lib/strategy/s3_zip_content.js
@@ -21,7 +21,6 @@ const crypto = require('crypto');
 const fs = require('fs');
 const AWS = require('aws-sdk');
 const BbPromise = require('bluebird');
-const shellescape = require('shell-escape');
 
 class S3ZipContent {
   constructor(strategy, options) {
@@ -56,9 +55,8 @@ class S3ZipContent {
             Expires: options.expires,
           });
 
-          // fixme: unescaped url in pkg/utils/kubelessutil.go:82
           resolve({
-            content: shellescape([url]),
+            content: url,
             checksum: `sha256:${shasum.digest('hex')}`,
             contentType: 'url+zip',
           });

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "serverless-kubeless",
-  "version": "0.9.1",
+  "version": "0.9.2",
   "description": "This plugin enables support for Kubeless within the [Serverless Framework](https://github.com/serverless).",
   "main": "index.js",
   "directories": {
@@ -29,8 +29,7 @@
     "jszip": "^3.1.3",
     "kubernetes-client": "^3.12.0",
     "lodash": "^4.17.4",
-    "moment": "^2.18.1",
-    "shell-escape": "^0.2.0"
+    "moment": "^2.18.1"
   },
   "devDependencies": {
     "chai": "^4.0.2",