Skip to content

Conversation

@EugeniyKiyashko
Copy link
Contributor

No description provided.

@EugeniyKiyashko EugeniyKiyashko self-assigned this Jan 7, 2026
@EugeniyKiyashko EugeniyKiyashko changed the title Demos: fix regular expression injection Demos test helper: fix regular expression injection Jan 7, 2026
@EugeniyKiyashko EugeniyKiyashko marked this pull request as ready for review January 7, 2026 12:57
Copilot AI review requested due to automatic review settings January 7, 2026 12:57
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request fixes a regular expression injection vulnerability in the demos test helper by properly escaping special characters before using user input to construct regular expressions.

Key changes:

  • Added escapeRegExp function to sanitize input strings
  • Modified the wrap function to escape strings before passing them to RegExp constructor

pharret31
pharret31 previously approved these changes Jan 7, 2026
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: EugeniyKiyashko <EugeniyKiyashko@users.noreply.github.com>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@EugeniyKiyashko EugeniyKiyashko merged commit c3dfa4e into DevExpress:26_1 Jan 8, 2026
115 of 117 checks passed
EugeniyKiyashko added a commit to EugeniyKiyashko/DevExtreme that referenced this pull request Jan 8, 2026
Signed-off-by: EugeniyKiyashko <EugeniyKiyashko@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
EugeniyKiyashko added a commit that referenced this pull request Jan 8, 2026
Signed-off-by: EugeniyKiyashko <EugeniyKiyashko@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants