scratch-storage 2.3.1 → 2.3.3

package/.github/CODEOWNERS.md

@@ -0,0 +1 @@
+@scratchfoundation/scratch-engineering

package/.github/workflows/ci-cd.yml

@@ -0,0 +1,51 @@
+name: CI/CD
+
+on:
+    workflow_dispatch: # Allows you to run this workflow manually from the Actions tab
+    push: # Runs whenever a commit is pushed to the repository
+
+concurrency:
+    group: "${{ github.workflow }} @ ${{ github.event.pull_request.head.label || github.head_ref || github.ref }}"
+    cancel-in-progress: true
+
+permissions:
+    contents: write # publish a GitHub release
+    pages: write # deploy to GitHub Pages
+    issues: write # comment on released issues
+    pull-requests: write # comment on released pull requests
+
+jobs:
+    ci-cd:
+        runs-on: ubuntu-latest
+        steps:
+            - uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4
+
+            - uses: wagoid/commitlint-github-action@5ce82f5d814d4010519d15f0552aec4f17a1e1fe # v5
+              if: github.event_name == 'pull_request'
+
+            - uses: actions/setup-node@1a4442cacd436585916779262731d5b162bc6ec7 # v3
+              with:
+                  cache: "npm"
+                  node-version-file: ".nvmrc"
+
+            - name: Info
+              run: |
+                  cat <<EOF
+                  Node version: $(node --version)
+                  NPM version: $(npm --version)
+                  GitHub ref: ${{ github.ref }}
+                  GitHub head ref: ${{ github.head_ref }}
+                  EOF
+
+            - name: Setup & Test
+              run: |
+                  npm ci
+                  npm run build
+                  npm run test
+
+            - name: Semantic release (configured to run dry if branch is other than 'develop')
+              env:
+                  NPM_TOKEN: ${{ secrets.NPM_TOKEN }}
+                  GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+              run: |
+                  npx --no -- semantic-release

package/.github/workflows/commitlint.yml

@@ -0,0 +1,12 @@
+name: Lint commit messages
+on: [pull_request]
+
+concurrency:
+  group: "${{ github.workflow }} @ ${{ github.event.pull_request.head.label || github.head_ref || github.sha }}"
+
+jobs:
+  commitlint:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4
+      - uses: wagoid/commitlint-github-action@5ce82f5d814d4010519d15f0552aec4f17a1e1fe # v5

package/.husky/.gitattributes

@@ -0,0 +1 @@
+* text eol=lf

package/.husky/commit-msg

@@ -0,0 +1,4 @@
+#!/usr/bin/env sh
+. "$(dirname -- "$0")/_/husky.sh"
+
+npx --no-install commitlint --edit "$1"

package/.nvmrc

@@ -0,0 +1 @@
+v16

package/CHANGELOG.md

@@ -0,0 +1,18 @@
+# Changelog
+
+All notable changes to this project will be documented in this file. See
+[Conventional Commits](https://conventionalcommits.org) for commit guidelines.
+
+## [2.3.3](https://github.com/scratchfoundation/scratch-storage/compare/v2.3.2...v2.3.3) (2024-02-22)
+
+
+### Bug Fixes
+
+* **deps:** update dependency scratch-semantic-release-config to v1.0.11 ([3d12e5d](https://github.com/scratchfoundation/scratch-storage/commit/3d12e5d60595254918f926f510623ed4ea2cc2b5))
+
+## [2.3.2](https://github.com/scratchfoundation/scratch-storage/compare/v2.3.1...v2.3.2) (2024-02-21)
+
+
+### Bug Fixes
+
+* **deps:** lock file maintenance ([ca51101](https://github.com/scratchfoundation/scratch-storage/commit/ca511019f5caddf8bc540a12ebfc808bb75973d0))

package/commitlint.config.js

@@ -1 +1,4 @@
-module.exports = {extends: ['@commitlint/config-conventional']};
+module.exports = {
+    extends: ['@commitlint/config-conventional'],
+    ignores: [message => message.startsWith('chore(release):')]
+};

package/dist/node/{8c63ecb0448dfbb8e804.worker.js → ce8a01e629587e4f90e4.worker.js}

@@ -94,11 +94,12 @@
 /***/ (function(module, exports, __webpack_require__) {
 
 /* eslint-env worker */
-var crossFetch = __webpack_require__(/*! cross-fetch */ "./node_modules/cross-fetch/dist/node-ponyfill.js").default;
 
+var crossFetch = __webpack_require__(/*! cross-fetch */ "./node_modules/cross-fetch/dist/node-ponyfill.js").default;
 var jobsActive = 0;
 var complete = [];
 var intervalId = null;
+
 /**
  * Register a step function.
  *
@@ -106,13 +107,13 @@ var intervalId = null;
  * parent. Then it checks the jobs count. If there are no further jobs, clear
  * the step.
  */
-
 var registerStep = function registerStep() {
   intervalId = setInterval(function () {
     if (complete.length) {
       // Send our chunk of completed requests and instruct postMessage to
       // transfer the buffers instead of copying them.
-      postMessage(complete.slice(), // Instruct postMessage that these buffers in the sent message
+      postMessage(complete.slice(),
+      // Instruct postMessage that these buffers in the sent message
       // should use their Transferable trait. After the postMessage
       // call the "buffers" will still be in complete if you looked,
       // but they will all be length 0 as the data they reference has
@@ -124,26 +125,22 @@ var registerStep = function registerStep() {
       }).filter(Boolean));
       complete.length = 0;
     }
-
     if (jobsActive === 0) {
       clearInterval(intervalId);
       intervalId = null;
     }
   }, 1);
 };
+
 /**
  * Receive a job from the parent and fetch the requested data.
  * @param {object} options.job A job id, url, and options descriptor to perform.
  */
-
-
 var onMessage = function onMessage(_ref) {
   var job = _ref.data;
-
   if (jobsActive === 0 && !intervalId) {
     registerStep();
   }
-
   jobsActive++;
   crossFetch(job.url, job.options).then(function (result) {
     if (result.ok) return result.arrayBuffer();
@@ -162,9 +159,9 @@ var onMessage = function onMessage(_ref) {
   }).then(function () {
     return jobsActive--;
   });
-}; // crossFetch means "fetch" is now always supported
-
+};
 
+// crossFetch means "fetch" is now always supported
 postMessage({
   support: {
     fetch: true
@@ -211,7 +208,7 @@ exports.default = fetch
 /*!***********************************************!*\
   !*** ./node_modules/node-fetch/lib/index.mjs ***!
   \***********************************************/
-/*! exports provided: default, Headers, Request, Response, FetchError */
+/*! exports provided: default, Headers, Request, Response, FetchError, AbortError */
 /***/ (function(__webpack_module__, __webpack_exports__, __webpack_require__) {
 
 "use strict";
@@ -220,6 +217,7 @@ __webpack_require__.r(__webpack_exports__);
 /* harmony export (binding) */ __webpack_require__.d(__webpack_exports__, "Request", function() { return Request; });
 /* harmony export (binding) */ __webpack_require__.d(__webpack_exports__, "Response", function() { return Response; });
 /* harmony export (binding) */ __webpack_require__.d(__webpack_exports__, "FetchError", function() { return FetchError; });
+/* harmony export (binding) */ __webpack_require__.d(__webpack_exports__, "AbortError", function() { return AbortError; });
 /* harmony import */ var stream__WEBPACK_IMPORTED_MODULE_0__ = __webpack_require__(/*! stream */ "stream");
 /* harmony import */ var http__WEBPACK_IMPORTED_MODULE_1__ = __webpack_require__(/*! http */ "http");
 /* harmony import */ var url__WEBPACK_IMPORTED_MODULE_2__ = __webpack_require__(/*! url */ "url");
@@ -1587,10 +1585,6 @@ function getNodeRequestOptions(request) {
 		agent = agent(parsedURL);
 	}
 
-	if (!headers.has('Connection') && !agent) {
-		headers.set('Connection', 'close');
-	}
-
 	// HTTP-network fetch step 4.2
 	// chunked encoding is handled by Node.js
 
@@ -1639,6 +1633,20 @@ const isDomainOrSubdomain = function isDomainOrSubdomain(destination, original)
 	return orig === dest || orig[orig.length - dest.length - 1] === '.' && orig.endsWith(dest);
 };
 
+/**
+ * isSameProtocol reports whether the two provided URLs use the same protocol.
+ *
+ * Both domains must already be in canonical form.
+ * @param {string|URL} original
+ * @param {string|URL} destination
+ */
+const isSameProtocol = function isSameProtocol(destination, original) {
+	const orig = new URL$1(original).protocol;
+	const dest = new URL$1(destination).protocol;
+
+	return orig === dest;
+};
+
 /**
  * Fetch function
  *
@@ -1670,7 +1678,7 @@ function fetch(url, opts) {
 			let error = new AbortError('The user aborted a request.');
 			reject(error);
 			if (request.body && request.body instanceof stream__WEBPACK_IMPORTED_MODULE_0__.Readable) {
-				request.body.destroy(error);
+				destroyStream(request.body, error);
 			}
 			if (!response || !response.body) return;
 			response.body.emit('error', error);
@@ -1711,9 +1719,43 @@ function fetch(url, opts) {
 
 		req.on('error', function (err) {
 			reject(new FetchError(`request to ${request.url} failed, reason: ${err.message}`, 'system', err));
+
+			if (response && response.body) {
+				destroyStream(response.body, err);
+			}
+
 			finalize();
 		});
 
+		fixResponseChunkedTransferBadEnding(req, function (err) {
+			if (signal && signal.aborted) {
+				return;
+			}
+
+			if (response && response.body) {
+				destroyStream(response.body, err);
+			}
+		});
+
+		/* c8 ignore next 18 */
+		if (parseInt(process.version.substring(1)) < 14) {
+			// Before Node.js 14, pipeline() does not fully support async iterators and does not always
+			// properly handle when the socket close/end events are out of order.
+			req.on('socket', function (s) {
+				s.addListener('close', function (hadError) {
+					// if a data listener is still present we didn't end cleanly
+					const hasDataListener = s.listenerCount('data') > 0;
+
+					// if end happened before close but the socket didn't emit an error, do it now
+					if (response && hasDataListener && !hadError && !(signal && signal.aborted)) {
+						const err = new Error('Premature close');
+						err.code = 'ERR_STREAM_PREMATURE_CLOSE';
+						response.body.emit('error', err);
+					}
+				});
+			});
+		}
+
 		req.on('response', function (res) {
 			clearTimeout(reqTimeout);
 
@@ -1785,7 +1827,7 @@ function fetch(url, opts) {
 							size: request.size
 						};
 
-						if (!isDomainOrSubdomain(request.url, locationURL)) {
+						if (!isDomainOrSubdomain(request.url, locationURL) || !isSameProtocol(request.url, locationURL)) {
 							for (const name of ['authorization', 'www-authenticate', 'cookie', 'cookie2']) {
 								requestOpts.headers.delete(name);
 							}
@@ -1878,6 +1920,13 @@ function fetch(url, opts) {
 					response = new Response(body, response_options);
 					resolve(response);
 				});
+				raw.on('end', function () {
+					// some old IIS servers return zero-length OK deflate responses, so 'data' is never emitted.
+					if (!response) {
+						response = new Response(body, response_options);
+						resolve(response);
+					}
+				});
 				return;
 			}
 
@@ -1897,6 +1946,44 @@ function fetch(url, opts) {
 		writeToStream(req, request);
 	});
 }
+function fixResponseChunkedTransferBadEnding(request, errorCallback) {
+	let socket;
+
+	request.on('socket', function (s) {
+		socket = s;
+	});
+
+	request.on('response', function (response) {
+		const headers = response.headers;
+
+		if (headers['transfer-encoding'] === 'chunked' && !headers['content-length']) {
+			response.once('close', function (hadError) {
+				// tests for socket presence, as in some situations the
+				// the 'socket' event is not triggered for the request
+				// (happens in deno), avoids `TypeError`
+				// if a data listener is still present we didn't end cleanly
+				const hasDataListener = socket && socket.listenerCount('data') > 0;
+
+				if (hasDataListener && !hadError) {
+					const err = new Error('Premature close');
+					err.code = 'ERR_STREAM_PREMATURE_CLOSE';
+					errorCallback(err);
+				}
+			});
+		}
+	});
+}
+
+function destroyStream(stream, err) {
+	if (stream.destroy) {
+		stream.destroy(err);
+	} else {
+		// node < 8
+		stream.emit('error', err);
+		stream.end();
+	}
+}
+
 /**
  * Redirect code matching
  *
@@ -4184,4 +4271,4 @@ module.exports = require("zlib");
 /***/ })
 
 /******/ });
-//# sourceMappingURL=8c63ecb0448dfbb8e804.worker.js.map
+//# sourceMappingURL=ce8a01e629587e4f90e4.worker.js.map