Page MenuHomePhabricator

Craft more mediawiki-specific and php semgrep rule sets
Closed, DeclinedPublic8 Estimated Story Points

Description

Prior work in this space: T335892#9936225. The various TPR rules could easily be modified to find http leaks, etc. within Wikimedia codebases.

Event Timeline

sbassett changed the task status from Open to In Progress.Oct 3 2023, 4:58 PM
sbassett claimed this task.
sbassett triaged this task as Medium priority.
sbassett moved this task from Incoming to In Progress on the Security-Team board.
sbassett moved this task from Backlog to In Progress on the user-sbassett board.
sbassett set the point value for this task to 8.Oct 3 2023, 5:00 PM
sbassett set Due Date to Dec 29 2023, 6:00 AM.
sbassett removed a project: user-sbassett.

@sbassett: Hi, the Due Date set for this open task passed a while ago.
Could you please either update or reset the Due Date (by clicking Edit Task), or set the status of this task to resolved in case this task is done? Thanks!

sbassett changed Due Date from Dec 29 2023, 6:00 AM to Jun 30 2024, 5:00 AM.Apr 16 2024, 2:31 PM

Write-up of some of this quarter's work is here: T335892#9936225

@sbassett: Hi, the Due Date set for this open task passed a while ago.
Could you please either update or reset the Due Date (by clicking Edit Task), or set the status of this task to resolved in case this task is done? Thanks!

sbassett changed Due Date from Jun 30 2024, 5:00 AM to Sep 27 2024, 5:00 AM.Aug 26 2024, 2:34 PM

To split this up (discussion between me and @sbassett)

  1. Research the existing PHP rulesets that are in semgrep - default rules
  2. Research historical vulnerabilities and what would be a good candidate for a new rule - e.g. message API (but in general there's a lot of good coverage there, but would be good to find these issues earlier), permissions layer (however this might be tricky), csrf
  3. Review the rules we already contracted semgrep to write - semgrep functions that are potentially ones that can be used incorrect - can we refine which ones are most effective and less false positives, or use as inspiration for more? https://gitlab.wikimedia.org/repos/security/wikimedia-semgrep-rules especially https://gitlab.wikimedia.org/repos/security/wikimedia-semgrep-rules/-/blob/main/php/mw-php-sec-sniff.yaml
Aklapper changed the task status from In Progress to Open.Apr 11 2025, 10:14 PM

Resetting task status from "In Progress" to "Open" as this task has been "in progress" for more than one and a half years (see T380300).

Aklapper removed Due Date which was set to Sep 27 2024, 5:00 AM.Nov 26 2025, 11:39 PM
Aklapper removed a subscriber: acooper.

Reset Due Date, which was long ago.