update deps

node_modules/aws4/.travis.yml

@@ -2,4 +2,8 @@ language: node_js
 node_js:
   - "0.10"
   - "0.12"
-  - "4.2"
+  - "4"
+  - "6"
+  - "8"
+  - "10"
+  - "12"

node_modules/aws4/aws4.js

@@ -220,12 +220,22 @@ RequestSigner.prototype.canonicalString = function() {
   }
 
   if (query) {
-    queryStr = encodeRfc3986(querystring.stringify(Object.keys(query).sort().reduce(function(obj, key) {
+    var reducedQuery = Object.keys(query).reduce(function(obj, key) {
       if (!key) return obj
       obj[key] = !Array.isArray(query[key]) ? query[key] :
         (firstValOnly ? query[key][0] : query[key].slice().sort())
       return obj
-    }, {})))
+    }, {})
+    var encodedQueryPieces = []
+    Object.keys(reducedQuery).forEach(function(key) {
+      var encodedPrefix = encodeURIComponent(key) + '='
+      if (!Array.isArray(reducedQuery[key])) {
+        encodedQueryPieces.push(encodeRfc3986(encodedPrefix + encodeURIComponent(reducedQuery[key])))
+      } else {
+        reducedQuery[key].forEach(function(val) { encodedQueryPieces.push(encodeRfc3986(encodedPrefix + encodeURIComponent(val))) })
+      }
+    })
+    queryStr = encodedQueryPieces.sort().join('&')
   }
   if (pathStr !== '/') {
     if (normalizePath) pathStr = pathStr.replace(/\/{2,}/g, '/')
@@ -233,7 +243,7 @@ RequestSigner.prototype.canonicalString = function() {
       if (normalizePath && piece === '..') {
         path.pop()
       } else if (!normalizePath || piece !== '.') {
-        if (decodePath) piece = decodeURIComponent(piece)
+        if (decodePath) piece = decodeURIComponent(piece).replace(/\+/g, ' ')
         path.push(encodeRfc3986(encodeURIComponent(piece)))
       }
       return path
@@ -289,8 +299,16 @@ RequestSigner.prototype.defaultCredentials = function() {
 }
 
 RequestSigner.prototype.parsePath = function() {
-  var path = this.request.path || '/',
-      queryIx = path.indexOf('?'),
+  var path = this.request.path || '/'
+
+  // S3 doesn't always encode characters > 127 correctly and
+  // all services don't encode characters > 255 correctly
+  // So if there are non-reserved chars (and it's not already all % encoded), just encode them all
+  if (/[^0-9A-Za-z;,/?:@&=+$\-_.!~*'()#%]/.test(path)) {
+    path = encodeURI(decodeURI(path))
+  }
+
+  var queryIx = path.indexOf('?'),
       query = null
 
   if (queryIx >= 0) {
@@ -298,15 +316,6 @@ RequestSigner.prototype.parsePath = function() {
     path = path.slice(0, queryIx)
   }
 
-  // S3 doesn't always encode characters > 127 correctly and
-  // all services don't encode characters > 255 correctly
-  // So if there are non-reserved chars (and it's not already all % encoded), just encode them all
-  if (/[^0-9A-Za-z!'()*\-._~%/]/.test(path)) {
-    path = path.split('/').map(function(piece) {
-      return encodeURIComponent(decodeURIComponent(piece))
-    }).join('/')
-  }
-
   this.parsedPath = {
     path: path,
     query: query,

node_modules/aws4/package.json

@@ -1,8 +1,8 @@
 {
   "_from": "aws4@^1.8.0",
-  "_id": "aws4@1.8.0",
+  "_id": "aws4@1.9.0",
   "_inBundle": false,
-  "_integrity": "sha512-ReZxvNHIOv88FlT7rxcXIIC0fPt4KZqZbOlivyWtXLt8ESx84zd3kMC6iK5jVeS2qt+g7ftS7ye4fi06X5rtRQ==",
+  "_integrity": "sha512-Uvq6hVe90D0B2WEnUqtdgY1bATGz3mw33nH9Y+dmA+w5DHvUmBgkr5rM/KCHpCsiFNRUfokW/szpPPgMK2hm4A==",
   "_location": "/aws4",
   "_phantomChildren": {},
   "_requested": {
@@ -18,10 +18,10 @@
   "_requiredBy": [
     "/request"
   ],
-  "_resolved": "https://registry.npmjs.org/aws4/-/aws4-1.8.0.tgz",
-  "_shasum": "f0e003d9ca9e7f59c7a508945d7b2ef9a04a542f",
+  "_resolved": "https://registry.npmjs.org/aws4/-/aws4-1.9.0.tgz",
+  "_shasum": "24390e6ad61386b0a747265754d2a17219de862c",
   "_spec": "aws4@^1.8.0",
-  "_where": "F:\\projects\\vanillajs-seed\\node_modules\\request",
+  "_where": "/home/s2/Code/vanillajs-seed/node_modules/request",
   "author": {
     "name": "Michael Hart",
     "email": "michael.hart.au@gmail.com",
@@ -98,7 +98,7 @@
     "url": "git+https://github.com/mhart/aws4.git"
   },
   "scripts": {
-    "test": "mocha ./test/fast.js ./test/slow.js -b -t 100s -R list"
+    "test": "mocha ./test/fast.js -b -t 100s -R list"
   },
-  "version": "1.8.0"
+  "version": "1.9.0"
 }