Address unsafe regular expression in checkuser/cidr.js
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	Dreamy_Jazz
	May 5 2023, 11:41 AM

Description

The mwgate-node16-docker job is raising the following warning:

21:10:37 /src/modules/ext.checkUser/checkuser/cidr.js
21:10:37   61:29  warning  Unsafe Regular Expression  security/detect-unsafe-regex

Ideally this regex should be fixed.

	Subject	Repo	Branch	Lines +/-
	CIDRCalculator: replace unsafe regex with split()	mediawiki/extensions/CheckUser	master	+21 -22

Dreamy_Jazz updated the task description. (Show Details)

Dreamy_Jazz triaged this task as Low priority.May 5 2023, 11:44 AM

Change 976879 had a related patch set uploaded (by DatGuy; author: DatGuy):

[mediawiki/extensions/CheckUser@master] CIDRCalculator: replace unsafe regex with split()

DatGuy subscribed.

Change 976879 merged by jenkins-bot:

[mediawiki/extensions/CheckUser@master] CIDRCalculator: replace unsafe regex with split()