Skip to content
This repository was archived by the owner on Dec 9, 2024. It is now read-only.

Commit 44b5217

Browse files
authored
remove shell-escaping url in S3ZipContent (#200)
1 parent b037631 commit 44b5217

File tree

4 files changed

+6
-14
lines changed

4 files changed

+6
-14
lines changed

.travis.yml

+2-2
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,8 @@ services:
99
- docker
1010
env:
1111
global:
12-
- KUBELESS_VERSION: v1.0.0-alpha.8
13-
- KUBELESS_KAFKA_VERSION: v1.0.0-beta.0
12+
- KUBELESS_VERSION: v1.0.6
13+
- KUBELESS_KAFKA_VERSION: v1.0.2
1414
- MINIKUBE_VERSION: v0.25.2
1515
- REPO_DOMAIN: serverless
1616
- REPO_NAME: serverless-kubeless

lib/strategy/s3_zip_content.js

+1-3
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@ const crypto = require('crypto');
2121
const fs = require('fs');
2222
const AWS = require('aws-sdk');
2323
const BbPromise = require('bluebird');
24-
const shellescape = require('shell-escape');
2524

2625
class S3ZipContent {
2726
constructor(strategy, options) {
@@ -56,9 +55,8 @@ class S3ZipContent {
5655
Expires: options.expires,
5756
});
5857

59-
// fixme: unescaped url in pkg/utils/kubelessutil.go:82
6058
resolve({
61-
content: shellescape([url]),
59+
content: url,
6260
checksum: `sha256:${shasum.digest('hex')}`,
6361
contentType: 'url+zip',
6462
});

package-lock.json

+1-6
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

+2-3
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "serverless-kubeless",
3-
"version": "0.9.1",
3+
"version": "0.9.2",
44
"description": "This plugin enables support for Kubeless within the [Serverless Framework](https://github.com/serverless).",
55
"main": "index.js",
66
"directories": {
@@ -29,8 +29,7 @@
2929
"jszip": "^3.1.3",
3030
"kubernetes-client": "^3.12.0",
3131
"lodash": "^4.17.4",
32-
"moment": "^2.18.1",
33-
"shell-escape": "^0.2.0"
32+
"moment": "^2.18.1"
3433
},
3534
"devDependencies": {
3635
"chai": "^4.0.2",

0 commit comments

Comments
 (0)