Skip to content

Fix DS126858 false positive on --nomd5 and similar disable flags - #722

Merged
Giulia Stocco (gfs) merged 6 commits into
mainfrom
copilot/fix-ds126858-flagging
Feb 17, 2026
Merged

Fix DS126858 false positive on --nomd5 and similar disable flags#722
Giulia Stocco (gfs) merged 6 commits into
mainfrom
copilot/fix-ds126858-flagging

Conversation

Copilot AI commented Feb 4, 2026

Copy link
Copy Markdown
Contributor
  • Understand the current rule pattern and why it's matching --nomd5
  • Update DS126858 rule pattern to use negative lookbehind to avoid matching MD5 when preceded by "no", "no-", "no_", or "disable_"
  • Add test cases to must-not-match to validate --nomd5 and similar cases
  • Run self-tests to verify the fix works correctly
  • Validate that existing must-match cases still work
  • Add word boundaries to negative lookbehind patterns to prevent edge cases like xnomd5
  • Revert .gitignore change that added DevSkim-DotNet/nuget.config
  • Add changelog entry for version 1.0.72
Original prompt

This section details on the original issue you should resolve

<issue_title>[False Positive] DS126858 flagging --nomd5</issue_title>
<issue_description>## DS126858

What the rule matched

makeself --sha256 --nomd5

Why should this not be matched?

makeself uses md5 checksums as a basic verification mechanism, I am trying to change it to a sha256 checksum. From the makeself documentation:

--sha256 : Adds a SHA256 checksum for the archive. This is in addition to the MD5 / CRC checksums unless --nomd5 is also used.

https://github.com/megastep/makeself</issue_description>

<agent_instructions>Try to improve the md5 rule to be less broad. It seems like it probably shouldn't capture this 'nomd5' case.</agent_instructions>

Comments on the Issue (you are Copilot in this section)

Giulia Stocco (@gfs) Thanks for your report. I'll take another look at the md5 rule and see if I can narrow it down a bit.

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI and others added 2 commits February 4, 2026 23:17
Co-authored-by: gfs <98900+gfs@users.noreply.github.com>
Co-authored-by: gfs <98900+gfs@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix false positive on DS126858 with --nomd5 Fix DS126858 false positive on --nomd5 and similar disable flags Feb 4, 2026

@gfs Giulia Stocco (gfs) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment thread .gitignore Outdated
Co-authored-by: gfs <98900+gfs@users.noreply.github.com>
@gfs

Copy link
Copy Markdown
Contributor

Copilot please add a changelog entry for these changes

Co-authored-by: gfs <98900+gfs@users.noreply.github.com>

Copilot AI commented Feb 12, 2026

Copy link
Copy Markdown
Contributor Author

Copilot please add a changelog entry for these changes

Added changelog entry for version 1.0.72 in commit c19e2a3

@gfs
Giulia Stocco (gfs) marked this pull request as ready for review February 12, 2026 21:26
@gfs
Giulia Stocco (gfs) merged commit 9e483f7 into main Feb 17, 2026
19 checks passed
@gfs
Giulia Stocco (gfs) deleted the copilot/fix-ds126858-flagging branch February 17, 2026 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[False Positive] DS126858 flagging --nomd5

3 participants