Information Required ?
avatar PhilETaylor
PhilETaylor
18 Jan 2021

Steps to reproduce the issue

npm i

Expected result

No security issues

Actual result

node_modules/urijs

1 moderate severity vulnerability

To address all issues, run:
  npm audit fix

urijs  <=1.19.3
Severity: moderate
Hostname spoofing via backslashes in URL - https://npmjs.com/advisories/1595

System information (as much as possible)

macOS

Additional comments

npm audit fix fixes the problem but increments the lockfileVersion to 2 and touches almost every line so probably not what you want me to submit as a PR... maybe someone more experienced in npm can advise.

avatar PhilETaylor PhilETaylor - open - 18 Jan 2021
avatar joomla-cms-bot joomla-cms-bot - change - 18 Jan 2021
Title
[4] "1 moderate severity vulnerability" urijs <=1.19.3
[4] "1 moderate severity vulnerability" urijs
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 18 Jan 2021
avatar PhilETaylor PhilETaylor - change - 18 Jan 2021
The description was changed
avatar PhilETaylor PhilETaylor - edited - 18 Jan 2021
avatar brianteeman
brianteeman - comment - 18 Jan 2021
  1. We need a proper system to deal with these - they happen all the time and we shouldnt be doing this manually
  2. Its not audit fix that is changing it to lockfile 2 but your version of node
avatar PhilETaylor
PhilETaylor - comment - 18 Jan 2021

node -v = v15.6.0

I.e "up to date" :-)

avatar brianteeman
brianteeman - comment - 18 Jan 2021

Exactly but until this is changed

  },
  "engines": {
    "node": ">=10.19",
    "npm": ">=6.13.4"
  },
avatar wilsonge
wilsonge - comment - 21 Jan 2021

Technically it's the npm version rather than node. npm 7 contains the new package lock (but is shipping with node 15 by default). If you swap back to the npm LTS version (6.14.11) then you'll get package v1. And yes indeed we should update package.json to stop people trying with npm 7 for now until npm 7 goes stable

avatar brianteeman
brianteeman - comment - 21 Jan 2021

my mistake confusing the node and npm as the cause

avatar wilsonge
wilsonge - comment - 22 Jan 2021

Fixed the package. I don't wanna upgrade my npm version to find out what breaks tonight having just merged the bs5 stuff. so leaving this open to cover that

avatar brianteeman
brianteeman - comment - 25 Jan 2021

I think this can be closed now @wilsonge

avatar Quy Quy - change - 12 Mar 2021
Labels Added: Information Required
avatar Quy Quy - labeled - 12 Mar 2021
avatar brianteeman
brianteeman - comment - 14 Mar 2021

This should be closed as it is resolved

avatar Quy Quy - change - 13 Apr 2021
Status New Closed
Closed_Date 0000-00-00 00:00:00 2021-04-13 23:47:44
Closed_By Quy
avatar Quy Quy - close - 13 Apr 2021

Add a Comment

Login with GitHub to post a comment