Add note about possible ReDoS#13123
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
There was a problem hiding this comment.
Pull request overview
Updates PowerShell documentation across multiple versions to add guidance on regular-expression ReDoS risks, and to refresh metadata/links so readers can find the new “Regular expression safety” content.
Changes:
- Adds ReDoS/regex safety notes to
about_Switch,about_Split, andSelect-String. - Adds a new Regular expression safety section to
about_Comparison_Operators(5.1, 7.4–7.7) and retargets link references. - Updates
ms.datefields and fixes some internal link reference numbering/targets.
Reviewed changes
Copilot reviewed 16 out of 16 changed files in this pull request and generated 10 comments.
Show a summary per file
| File | Description |
|---|---|
| reference/7.7/Microsoft.PowerShell.Core/About/about_Switch.md | Updates ms.date, fixes link refs, and adds a regex safety note pointing to comparison operators guidance. |
| reference/7.7/Microsoft.PowerShell.Core/About/about_Split.md | Updates ms.date, adds regex safety note, and normalizes “See also” links to reference-style. |
| reference/7.7/Microsoft.PowerShell.Core/About/about_Comparison_Operators.md | Updates ms.date, adds “Regular expression safety” section, and rewires link references. |
| reference/7.6/Microsoft.PowerShell.Core/About/about_Switch.md | Same as 7.7: metadata/link ref updates + regex safety note. |
| reference/7.6/Microsoft.PowerShell.Core/About/about_Split.md | Same as 7.7: metadata + regex safety note + link ref normalization. |
| reference/7.6/Microsoft.PowerShell.Core/About/about_Comparison_Operators.md | Same as 7.7: adds regex safety section and updates link refs. |
| reference/7.5/Microsoft.PowerShell.Core/About/about_Switch.md | Same as 7.7: metadata/link ref updates + regex safety note. |
| reference/7.5/Microsoft.PowerShell.Core/About/about_Split.md | Same as 7.7: metadata + regex safety note + link ref normalization. |
| reference/7.5/Microsoft.PowerShell.Core/About/about_Comparison_Operators.md | Same as 7.7: adds regex safety section and updates link refs. |
| reference/7.4/Microsoft.PowerShell.Core/About/about_Switch.md | Same as 7.7: metadata/link ref updates + regex safety note. |
| reference/7.4/Microsoft.PowerShell.Core/About/about_Split.md | Same as 7.7: metadata + regex safety note + link ref normalization. |
| reference/7.4/Microsoft.PowerShell.Core/About/about_Comparison_Operators.md | Fixes online version view, adds regex safety section, and updates link refs. |
| reference/5.1/Microsoft.PowerShell.Core/About/about_Switch.md | Same as 7.7: metadata/link ref updates + regex safety note. |
| reference/5.1/Microsoft.PowerShell.Core/About/about_Split.md | Same as 7.7: metadata + regex safety note + link ref normalization. |
| reference/5.1/Microsoft.PowerShell.Core/About/about_Comparison_Operators.md | Same as 7.7: adds regex safety section and updates link refs. |
| reference/7.4/Microsoft.PowerShell.Utility/Select-String.md | Updates ms.date and adds a regex safety note linking to comparison operators guidance. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
7a5e83b to
427a991
Compare
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
427a991 to
651e440
Compare
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
PR Summary
PR Checklist