protractor icon indicating copy to clipboard operation
protractor copied to clipboard

CVE-2021-3807 in ansi-regex

Open SymbioticKilla opened this issue 4 years ago • 9 comments

-- [email protected] +-- [email protected] | -- [email protected] | -- [email protected]

Hi,

is there any chance to update chalk V2+? The dropped has-ansi dependency.

Thank you!

SymbioticKilla avatar Sep 21 '21 10:09 SymbioticKilla

Good question for maintainers. @IgorMinar @dgp1130 @mgechev Do you have information about the next steps of maintain this package?

StanislavKharchenko avatar Sep 21 '21 11:09 StanislavKharchenko

The issue here is about a regex DDOS vulnerability. This is pretty low risk as Protractor is not intended for production use or running on untrusted inputs.

The fix seems pretty easy, as our usage of chalk is quite minimal and an update should be straightforward. The trickier part is that the repository is not currently in a releaseable state. I'm hoping to spend some time soon to clean things up, fix CI, and get things back into a good state. Once that happens, this should be an easy and uncontroversial fix.

dgp1130 avatar Sep 23 '21 01:09 dgp1130

@dgp1130 Thanks for the update! Do you have plans also to consider https://github.com/angular/protractor/pull/5516 and release a Protractor with w3c compliant Selenium? Hope that we won't back to v7 with deprecated control flow...

StanislavKharchenko avatar Sep 23 '21 08:09 StanislavKharchenko

I am facing a facing a high security issue and the affected component in [email protected]. -- [email protected] +-- [email protected] | -- [email protected] | -- [email protected]

How can I resolve this? @dgp1130

shubham0827 avatar Sep 29 '21 05:09 shubham0827

@shubham0827 I suggest to make own forks of Protractor and maintain by yourself. Due to lack of responses and activities from Angular team this is very huge risk for business to rely on official Angular solutions. There was 2 years no activities on repo and no any plan/roadmap.

StanislavKharchenko avatar Sep 29 '21 08:09 StanislavKharchenko

There was 2 years no activities on repo and no any plan/roadmap.

@StanislavKharchenko, see https://github.com/angular/protractor/issues/5502

alan-agius4 avatar Jan 28 '22 10:01 alan-agius4

@alan-agius4 In regards to #5502 there was final decision and agreement to continue Protractor in form of v6. But repository still abandoned.

StanislavKharchenko avatar Jan 28 '22 12:01 StanislavKharchenko

@StanislavKharchenko, there wasn't a final decision about that, in-fact the following is stated in the linked comment.

For point (3), we are exploring the possibility of a shared ownership of the project with other enterprise partners. This effort will keep Protractor going in the form of version 6

To my knowledge, there wasn't much interest from enterprise partners about shared ownership. Although probably @dgp1130 and @mgechev will know more about it.


Back to the original issue, the mentioned CVE doesn't seem to effect the [email protected] at least from the CVE report https://nvd.nist.gov/vuln/detail/CVE-2021-3807. In fact the effected versions are 3.0.0 up to 5.0.1 and 6.0.0 up to 6.0.1.

I also tried this out locally and NPM didn't report any vulnerability.

 npm i protractor@7
npm WARN deprecated [email protected]: request has been deprecated, see https://github.com/request/request/issues/3142
npm WARN deprecated [email protected]: this library is no longer supported
npm WARN deprecated [email protected]: Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.


+ [email protected]
added 151 packages from 190 contributors and audited 151 packages in 6.785s

4 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities

alan-agius4 avatar Jan 28 '22 13:01 alan-agius4

To my knowledge, there wasn't much interest from enterprise partners about shared ownership. Although probably @dgp1130 and @mgechev will know more about it.

@alan-agius4 This also not true. Me and not only contacted with Keen at first, then with Angular devrel and the last what I heard that Protractor shared ownership possibility is under consideration (we talked about this in the summer of 2021). I personally proposed help in upgrade and maintenance of Protractor (https://github.com/angular/protractor/pull/5516 here were first attempts). I don't know what happened with Angular team in general, but any proposals with Protractor continuation were rejected one by one. Finally, I decided to fix Protractor v6-ish in forked repo and use my own solution. And advised make the similar approach for everyone whose e2e business suffered from indecision and your (Angular team) inactions.

StanislavKharchenko avatar Jan 28 '22 13:01 StanislavKharchenko