$ date
--- stdout ---
Thu Nov 21 22:40:28 UTC 2024
--- end ---
$ git clone file:///srv/git/mediawiki-extensions-SVGEdit.git repo --depth=1 -b REL1_39
--- stderr ---
Cloning into 'repo'...
--- stdout ---
--- end ---
$ git config user.name libraryupgrader
--- stdout ---
--- end ---
$ git config user.email tools.libraryupgrader@tools.wmflabs.org
--- stdout ---
--- end ---
$ git submodule update --init
--- stdout ---
--- end ---
$ grr init
--- stdout ---
Installed commit-msg hook.
--- end ---
$ git show-ref refs/heads/REL1_39
--- stdout ---
7577f0ca398a70a27661e96585d0a18383a62f3b refs/heads/REL1_39
--- end ---
$ /usr/bin/npm audit --json
--- stdout ---
{
"auditReportVersion": 2,
"vulnerabilities": {
"cross-spawn": {
"name": "cross-spawn",
"severity": "high",
"isDirect": false,
"via": [
{
"source": 1100563,
"name": "cross-spawn",
"dependency": "cross-spawn",
"title": "Regular Expression Denial of Service (ReDoS) in cross-spawn",
"url": "https://github.com/advisories/GHSA-3xgq-45jj-v275",
"severity": "high",
"cwe": [
"CWE-1333"
],
"cvss": {
"score": 7.5,
"vectorString": "CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:N/I:N/A:H"
},
"range": ">=7.0.0 <7.0.5"
}
],
"effects": [],
"range": "7.0.0 - 7.0.4",
"nodes": [
"node_modules/cross-spawn"
],
"fixAvailable": true
}
},
"metadata": {
"vulnerabilities": {
"info": 0,
"low": 0,
"moderate": 0,
"high": 1,
"critical": 0,
"total": 1
},
"dependencies": {
"prod": 1,
"dev": 285,
"optional": 0,
"peer": 0,
"peerOptional": 0,
"total": 285
}
}
}
--- end ---
$ /usr/bin/composer install
--- stderr ---
No composer.lock file present. Updating dependencies to latest instead of installing from lock file. See https://getcomposer.org/install for more information.
Loading composer repositories with package information
Updating dependencies
Lock file operations: 15 installs, 0 updates, 0 removals
- Locking mediawiki/minus-x (1.1.1)
- Locking php-parallel-lint/php-console-color (v1.0.1)
- Locking php-parallel-lint/php-console-highlighter (v1.0.0)
- Locking php-parallel-lint/php-parallel-lint (v1.3.2)
- Locking psr/container (2.0.2)
- Locking symfony/console (v5.4.47)
- Locking symfony/deprecation-contracts (v3.5.0)
- Locking symfony/polyfill-ctype (v1.31.0)
- Locking symfony/polyfill-intl-grapheme (v1.31.0)
- Locking symfony/polyfill-intl-normalizer (v1.31.0)
- Locking symfony/polyfill-mbstring (v1.31.0)
- Locking symfony/polyfill-php73 (v1.31.0)
- Locking symfony/polyfill-php80 (v1.31.0)
- Locking symfony/service-contracts (v3.5.0)
- Locking symfony/string (v6.4.15)
Writing lock file
Installing dependencies from lock file (including require-dev)
Package operations: 15 installs, 0 updates, 0 removals
0 [>---------------------------] 0 [->--------------------------]
- Installing symfony/polyfill-mbstring (v1.31.0): Extracting archive
- Installing symfony/polyfill-intl-normalizer (v1.31.0): Extracting archive
- Installing symfony/polyfill-intl-grapheme (v1.31.0): Extracting archive
- Installing symfony/polyfill-ctype (v1.31.0): Extracting archive
- Installing symfony/string (v6.4.15): Extracting archive
- Installing symfony/deprecation-contracts (v3.5.0): Extracting archive
- Installing psr/container (2.0.2): Extracting archive
- Installing symfony/service-contracts (v3.5.0): Extracting archive
- Installing symfony/polyfill-php80 (v1.31.0): Extracting archive
- Installing symfony/polyfill-php73 (v1.31.0): Extracting archive
- Installing symfony/console (v5.4.47): Extracting archive
- Installing mediawiki/minus-x (1.1.1): Extracting archive
- Installing php-parallel-lint/php-console-color (v1.0.1): Extracting archive
- Installing php-parallel-lint/php-console-highlighter (v1.0.0): Extracting archive
- Installing php-parallel-lint/php-parallel-lint (v1.3.2): Extracting archive
0/15 [>---------------------------] 0%
15/15 [============================] 100%
4 package suggestions were added by new dependencies, use `composer suggest` to see details.
Generating autoload files
10 packages you are using are looking for funding.
Use the `composer fund` command to find out more!
--- stdout ---
--- end ---
$ /usr/bin/npm audit --json
--- stdout ---
{
"auditReportVersion": 2,
"vulnerabilities": {
"cross-spawn": {
"name": "cross-spawn",
"severity": "high",
"isDirect": false,
"via": [
{
"source": 1100563,
"name": "cross-spawn",
"dependency": "cross-spawn",
"title": "Regular Expression Denial of Service (ReDoS) in cross-spawn",
"url": "https://github.com/advisories/GHSA-3xgq-45jj-v275",
"severity": "high",
"cwe": [
"CWE-1333"
],
"cvss": {
"score": 7.5,
"vectorString": "CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:N/I:N/A:H"
},
"range": ">=7.0.0 <7.0.5"
}
],
"effects": [],
"range": "7.0.0 - 7.0.4",
"nodes": [
"node_modules/cross-spawn"
],
"fixAvailable": true
}
},
"metadata": {
"vulnerabilities": {
"info": 0,
"low": 0,
"moderate": 0,
"high": 1,
"critical": 0,
"total": 1
},
"dependencies": {
"prod": 1,
"dev": 285,
"optional": 0,
"peer": 0,
"peerOptional": 0,
"total": 285
}
}
}
--- end ---
Attempting to npm audit fix
$ /usr/bin/npm audit fix --dry-run --only=dev --json
--- stderr ---
npm WARN invalid config only="dev" set in command line options
npm WARN invalid config Must be one of: null, prod, production
npm WARN EBADENGINE Unsupported engine {
npm WARN EBADENGINE package: '@es-joy/jsdoccomment@0.20.1',
npm WARN EBADENGINE required: { node: '^12 || ^14 || ^16 || ^17' },
npm WARN EBADENGINE current: { node: 'v18.19.0', npm: '9.2.0' }
npm WARN EBADENGINE }
npm WARN EBADENGINE Unsupported engine {
npm WARN EBADENGINE package: 'eslint-plugin-jsdoc@37.9.7',
npm WARN EBADENGINE required: { node: '^12 || ^14 || ^16 || ^17' },
npm WARN EBADENGINE current: { node: 'v18.19.0', npm: '9.2.0' }
npm WARN EBADENGINE }
--- stdout ---
{
"added": 285,
"removed": 0,
"changed": 0,
"audited": 286,
"funding": 32,
"audit": {
"auditReportVersion": 2,
"vulnerabilities": {
"cross-spawn": {
"name": "cross-spawn",
"severity": "high",
"isDirect": false,
"via": [
{
"source": 1100563,
"name": "cross-spawn",
"dependency": "cross-spawn",
"title": "Regular Expression Denial of Service (ReDoS) in cross-spawn",
"url": "https://github.com/advisories/GHSA-3xgq-45jj-v275",
"severity": "high",
"cwe": [
"CWE-1333"
],
"cvss": {
"score": 7.5,
"vectorString": "CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:N/I:N/A:H"
},
"range": ">=7.0.0 <7.0.5"
}
],
"effects": [],
"range": "7.0.0 - 7.0.4",
"nodes": [
""
],
"fixAvailable": true
}
},
"metadata": {
"vulnerabilities": {
"info": 0,
"low": 0,
"moderate": 0,
"high": 1,
"critical": 0,
"total": 1
},
"dependencies": {
"prod": 1,
"dev": 285,
"optional": 0,
"peer": 0,
"peerOptional": 0,
"total": 285
}
}
}
}
--- end ---
{"added": 285, "removed": 0, "changed": 0, "audited": 286, "funding": 32, "audit": {"auditReportVersion": 2, "vulnerabilities": {"cross-spawn": {"name": "cross-spawn", "severity": "high", "isDirect": false, "via": [{"source": 1100563, "name": "cross-spawn", "dependency": "cross-spawn", "title": "Regular Expression Denial of Service (ReDoS) in cross-spawn", "url": "https://github.com/advisories/GHSA-3xgq-45jj-v275", "severity": "high", "cwe": ["CWE-1333"], "cvss": {"score": 7.5, "vectorString": "CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:N/I:N/A:H"}, "range": ">=7.0.0 <7.0.5"}], "effects": [], "range": "7.0.0 - 7.0.4", "nodes": [""], "fixAvailable": true}}, "metadata": {"vulnerabilities": {"info": 0, "low": 0, "moderate": 0, "high": 1, "critical": 0, "total": 1}, "dependencies": {"prod": 1, "dev": 285, "optional": 0, "peer": 0, "peerOptional": 0, "total": 285}}}}
$ /usr/bin/npm audit fix --only=dev
--- stderr ---
npm WARN invalid config only="dev" set in command line options
npm WARN invalid config Must be one of: null, prod, production
npm WARN EBADENGINE Unsupported engine {
npm WARN EBADENGINE package: '@es-joy/jsdoccomment@0.20.1',
npm WARN EBADENGINE required: { node: '^12 || ^14 || ^16 || ^17' },
npm WARN EBADENGINE current: { node: 'v18.19.0', npm: '9.2.0' }
npm WARN EBADENGINE }
npm WARN EBADENGINE Unsupported engine {
npm WARN EBADENGINE package: 'eslint-plugin-jsdoc@37.9.7',
npm WARN EBADENGINE required: { node: '^12 || ^14 || ^16 || ^17' },
npm WARN EBADENGINE current: { node: 'v18.19.0', npm: '9.2.0' }
npm WARN EBADENGINE }
--- stdout ---
added 285 packages, and audited 286 packages in 3s
32 packages are looking for funding
run `npm fund` for details
found 0 vulnerabilities
--- end ---
Verifying that tests still pass
$ /usr/bin/npm ci
--- stderr ---
npm WARN EBADENGINE Unsupported engine {
npm WARN EBADENGINE package: '@es-joy/jsdoccomment@0.20.1',
npm WARN EBADENGINE required: { node: '^12 || ^14 || ^16 || ^17' },
npm WARN EBADENGINE current: { node: 'v18.19.0', npm: '9.2.0' }
npm WARN EBADENGINE }
npm WARN EBADENGINE Unsupported engine {
npm WARN EBADENGINE package: 'eslint-plugin-jsdoc@37.9.7',
npm WARN EBADENGINE required: { node: '^12 || ^14 || ^16 || ^17' },
npm WARN EBADENGINE current: { node: 'v18.19.0', npm: '9.2.0' }
npm WARN EBADENGINE }
--- stdout ---
added 285 packages, and audited 286 packages in 3s
32 packages are looking for funding
run `npm fund` for details
found 0 vulnerabilities
--- end ---
$ /usr/bin/npm test
--- stdout ---
> test
> grunt test
Running "eslint:all" (eslint) task
/src/repo/modules/ext.svgedit.editButton.js
50:8 warning jQuery collection names must match the variablePattern no-jquery/variable-pattern
55:4 warning Avoid queries which search the entire DOM. Keep DOM nodes in memory where possible no-jquery/no-global-selector
/src/repo/modules/ext.svgedit.editor.js
15:3 warning Missing JSDoc @return declaration jsdoc/require-returns
19:0 warning Syntax error in namepath: options: jsdoc/valid-types
19:0 warning Expected @param names to be "options". Got "options:, options" jsdoc/check-param-names
19:0 warning Missing JSDoc @param "options:" type jsdoc/require-param-type
22:1 warning This line has a length of 107. Maximum allowed is 100 max-len
23:1 warning This line has a length of 143. Maximum allowed is 100 max-len
24:0 warning Missing JSDoc @param "options" type jsdoc/require-param-type
43:0 warning Missing JSDoc @param "options" type jsdoc/require-param-type
44:0 warning Missing JSDoc @param "imageinfo" type jsdoc/require-param-type
64:1 warning This line has a length of 104. Maximum allowed is 100 max-len
65:1 warning This line has a length of 107. Maximum allowed is 100 max-len
69:5 warning jQuery collection names must match the variablePattern no-jquery/variable-pattern
84:8 warning jQuery collection names must match the variablePattern no-jquery/variable-pattern
84:16 warning Avoid queries which search the entire DOM. Keep DOM nodes in memory where possible no-jquery/no-global-selector
85:4 warning Avoid queries which search the entire DOM. Keep DOM nodes in memory where possible no-jquery/no-global-selector
91:4 warning Avoid queries which search the entire DOM. Keep DOM nodes in memory where possible no-jquery/no-global-selector
91:4 warning Prefer DOM building to parsing HTML literals no-jquery/no-parse-html-literal
92:8 warning jQuery collection names must match the variablePattern no-jquery/variable-pattern
92:18 warning Avoid queries which search the entire DOM. Keep DOM nodes in memory where possible no-jquery/no-global-selector
100:6 warning Avoid queries which search the entire DOM. Keep DOM nodes in memory where possible no-jquery/no-global-selector
104:7 warning Avoid queries which search the entire DOM. Keep DOM nodes in memory where possible no-jquery/no-global-selector
114:6 warning Avoid queries which search the entire DOM. Keep DOM nodes in memory where possible no-jquery/no-global-selector
115:6 warning Avoid queries which search the entire DOM. Keep DOM nodes in memory where possible no-jquery/no-global-selector
125:6 warning Avoid queries which search the entire DOM. Keep DOM nodes in memory where possible no-jquery/no-global-selector
126:6 warning Avoid queries which search the entire DOM. Keep DOM nodes in memory where possible no-jquery/no-global-selector
130:4 warning Avoid queries which search the entire DOM. Keep DOM nodes in memory where possible no-jquery/no-global-selector
133:4 warning Avoid queries which search the entire DOM. Keep DOM nodes in memory where possible no-jquery/no-global-selector
136:4 warning Avoid queries which search the entire DOM. Keep DOM nodes in memory where possible no-jquery/no-global-selector
141:21 warning Avoid queries which search the entire DOM. Keep DOM nodes in memory where possible no-jquery/no-global-selector
148:9 warning Unexpected alert no-alert
151:9 warning Unexpected alert no-alert
157:4 warning Avoid queries which search the entire DOM. Keep DOM nodes in memory where possible no-jquery/no-global-selector
165:4 warning Avoid queries which search the entire DOM. Keep DOM nodes in memory where possible no-jquery/no-global-selector
167:20 warning A constructor name should not start with a lowercase letter new-cap
167:20 warning 'embedded_svg_edit' is not defined no-undef
180:10 warning Unexpected alert no-alert
201:8 warning jQuery collection names must match the variablePattern no-jquery/variable-pattern
204:11 warning Avoid queries which search the entire DOM. Keep DOM nodes in memory where possible no-jquery/no-global-selector
205:20 warning Avoid queries which search the entire DOM. Keep DOM nodes in memory where possible no-jquery/no-global-selector
/src/repo/modules/ext.svgedit.embedapi.js
55:1 warning This line has a length of 130. Maximum allowed is 100 max-len
58:1 warning Unexpected function declaration in the global scope, wrap in an IIFE for a local variable, assign as global property for a global variable no-implicit-globals
58:10 warning Identifier 'embedded_svg_edit' is not in camel case camelcase
63:16 warning Identifier 'embedded_svg_edit' is not in camel case camelcase
67:1 warning This line has a length of 119. Maximum allowed is 100 max-len
68:1 warning This line has a length of 397. Maximum allowed is 100 max-len
70:1 warning This line has a length of 156. Maximum allowed is 100 max-len
79:9 warning 't' is already declared in the upper scope on line 92 column 6 no-shadow
83:1 warning This line has a length of 124. Maximum allowed is 100 max-len
83:27 warning 'args' used outside of binding context block-scoped-var
91:1 warning This line has a length of 110. Maximum allowed is 100 max-len
99:27 warning eval can be harmful no-eval
105:1 warning This line has a length of 104. Maximum allowed is 100 max-len
109:1 warning Identifier 'embedded_svg_edit' is not in camel case camelcase
114:12 warning Avoid arguments.callee no-caller
122:5 warning Unexpected control character(s) in regular expression: \x00, \x1f no-control-regex
122:5 warning Unexpected combined character in character class no-misleading-character-class
122:9 warning Unnecessary escape character: \" no-useless-escape
142:1 warning Identifier 'embedded_svg_edit' is not in camel case camelcase
151:79 warning 'argstr' used outside of binding context block-scoped-var
/src/repo/modules/ext.svgedit.formmultipart.js
10:0 warning Missing JSDoc @param "fields" type jsdoc/require-param-type
13:1 warning Unexpected function declaration in the global scope, wrap in an IIFE for a local variable, assign as global property for a global variable no-implicit-globals
26:16 warning Do not access Object.prototype method 'hasOwnProperty' from target object no-prototype-builtins
/src/repo/modules/ext.svgedit.inline.js
21:30 warning Unnecessary escape character: \/ no-useless-escape
21:40 warning Unnecessary escape character: \/ no-useless-escape
37:8 warning jQuery collection names must match the variablePattern no-jquery/variable-pattern
45:3 warning Avoid queries which search the entire DOM. Keep DOM nodes in memory where possible no-jquery/no-global-selector
/src/repo/modules/ext.svgedit.io.js
10:5 warning Unexpected 'var' declaration in the global scope, wrap in an IIFE for a local variable, assign as global property for a global variable no-implicit-globals
14:0 warning Missing JSDoc @return type jsdoc/require-returns-type
24:0 warning The type 'imageinfo' is undefined jsdoc/no-undefined-types
36:4 warning Prefer Array#forEach to $.each no-jquery/no-each-util
51:0 warning The type 'xmlSource' is undefined jsdoc/no-undefined-types
51:0 warning The type 'textStatus' is undefined jsdoc/no-undefined-types
51:0 warning The type 'xhr' is undefined jsdoc/no-undefined-types
52:0 warning The type 'xhr' is undefined jsdoc/no-undefined-types
52:0 warning The type 'textStatus' is undefined jsdoc/no-undefined-types
52:0 warning The type 'errorThrown' is undefined jsdoc/no-undefined-types
70:0 warning The type 'xmlSource' is undefined jsdoc/no-undefined-types
70:0 warning The type 'textStatus' is undefined jsdoc/no-undefined-types
70:0 warning The type 'xhr' is undefined jsdoc/no-undefined-types
84:0 warning Syntax error in namepath: target: jsdoc/valid-types
84:0 warning Expected @param names to be "target, callback". Got "target:, target, callback" jsdoc/check-param-names
85:0 warning Missing JSDoc @param "target" type jsdoc/require-param-type
86:0 warning The type 'token' is undefined jsdoc/no-undefined-types
98:4 warning Prefer Array#forEach to $.each no-jquery/no-each-util
108:0 warning Syntax error in namepath: target: jsdoc/valid-types
108:0 warning Expected @param names to be "target, data, comment, callback". Got "target:, data:, comment:, callback:, target, data, comment, callback" jsdoc/check-param-names
109:0 warning Syntax error in namepath: data: jsdoc/valid-types
110:0 warning Syntax error in namepath: comment: jsdoc/valid-types
111:0 warning The type 'textStatus' is undefined jsdoc/no-undefined-types
111:0 warning The type 'xhr' is undefined jsdoc/no-undefined-types
111:0 warning Syntax error in namepath: callback: jsdoc/valid-types
112:0 warning Missing JSDoc @param "target" type jsdoc/require-param-type
113:0 warning Missing JSDoc @param "data" type jsdoc/require-param-type
114:0 warning Missing JSDoc @param "comment" type jsdoc/require-param-type
115:0 warning Missing JSDoc @param "callback" type jsdoc/require-param-type
119:24 warning 'FormMultipart' is not defined no-undef
137:6 warning Unexpected alert no-alert
/src/repo/modules/ext.svgedit.toolbar.js
35:5 warning jQuery collection names must match the variablePattern no-jquery/variable-pattern
39:25 warning 'filename' is already declared in the upper scope on line 34 column 8 no-shadow
52:2 warning Avoid queries which search the entire DOM. Keep DOM nodes in memory where possible no-jquery/no-global-selector
✖ 102 problems (0 errors, 102 warnings)
Running "banana:all" (banana) task
>> 1 message directory checked.
Done.
--- end ---
{"1100563": {"source": 1100563, "name": "cross-spawn", "dependency": "cross-spawn", "title": "Regular Expression Denial of Service (ReDoS) in cross-spawn", "url": "https://github.com/advisories/GHSA-3xgq-45jj-v275", "severity": "high", "cwe": ["CWE-1333"], "cvss": {"score": 7.5, "vectorString": "CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:N/I:N/A:H"}, "range": ">=7.0.0 <7.0.5"}}
Upgrading n:cross-spawn from 7.0.3 -> 7.0.6
$ package-lock-lint package-lock.json
--- stdout ---
Checking package-lock.json
--- end ---
build: Updating cross-spawn to 7.0.6
* https://github.com/advisories/GHSA-3xgq-45jj-v275
$ git add .
--- stdout ---
--- end ---
$ git commit -F /tmp/tmpi4cy2fy6
--- stdout ---
[REL1_39 5ed2434] build: Updating cross-spawn to 7.0.6
1 file changed, 6 insertions(+), 6 deletions(-)
--- end ---
$ git format-patch HEAD~1 --stdout
--- stdout ---
From 5ed24342f155dc1f650af01d25cb64f5bf883274 Mon Sep 17 00:00:00 2001
From: libraryupgrader <tools.libraryupgrader@tools.wmflabs.org>
Date: Thu, 21 Nov 2024 22:40:46 +0000
Subject: [PATCH] build: Updating cross-spawn to 7.0.6
* https://github.com/advisories/GHSA-3xgq-45jj-v275
Change-Id: I7c43c4f9645fbca77faa644ce38653d7c3fa9beb
---
package-lock.json | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/package-lock.json b/package-lock.json
index 75b0b9d..b1966d5 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -526,9 +526,9 @@
"dev": true
},
"node_modules/cross-spawn": {
- "version": "7.0.3",
- "resolved": "https://registry.npmjs.org/cross-spawn/-/cross-spawn-7.0.3.tgz",
- "integrity": "sha512-iRDPJKUPVEND7dHPO8rkbOnPpyDygcDFtWjpeWNCgy8WP2rXcxXL8TskReQl6OrB2G7+UJrags1q15Fudc7G6w==",
+ "version": "7.0.6",
+ "resolved": "https://registry.npmjs.org/cross-spawn/-/cross-spawn-7.0.6.tgz",
+ "integrity": "sha512-uV2QOWP2nWzsy2aMp8aRibhi9dlzF5Hgh5SHaB9OiTGEyDTiJJyx0uy51QXdyWbtAHNua4XJzUKca3OzKUd3vA==",
"dev": true,
"dependencies": {
"path-key": "^3.1.0",
@@ -3650,9 +3650,9 @@
"dev": true
},
"cross-spawn": {
- "version": "7.0.3",
- "resolved": "https://registry.npmjs.org/cross-spawn/-/cross-spawn-7.0.3.tgz",
- "integrity": "sha512-iRDPJKUPVEND7dHPO8rkbOnPpyDygcDFtWjpeWNCgy8WP2rXcxXL8TskReQl6OrB2G7+UJrags1q15Fudc7G6w==",
+ "version": "7.0.6",
+ "resolved": "https://registry.npmjs.org/cross-spawn/-/cross-spawn-7.0.6.tgz",
+ "integrity": "sha512-uV2QOWP2nWzsy2aMp8aRibhi9dlzF5Hgh5SHaB9OiTGEyDTiJJyx0uy51QXdyWbtAHNua4XJzUKca3OzKUd3vA==",
"dev": true,
"requires": {
"path-key": "^3.1.0",
--
2.39.2
--- end ---