-
-
Notifications
You must be signed in to change notification settings - Fork 510
Rewrite escape/unescape for SQL #1245
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
Review or Edit in CodeSandboxOpen the branch in Web Editor • VS Code • Insiders Open Preview |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit f0003b9:
|
cc3d619
to
566572b
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #1245 +/- ##
==========================================
- Coverage 80.59% 7.91% -72.69%
==========================================
Files 225 225
Lines 12424 12591 +167
Branches 1559 1565 +6
==========================================
- Hits 10013 996 -9017
- Misses 1669 11586 +9917
+ Partials 742 9 -733 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
1ae4589
to
f3082b7
Compare
f3082b7
to
6b5c131
Compare
…ing or encoding Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
This reverts commit 566572b.
6b5c131
to
f0003b9
Compare
Potential fix for https://github.com/ukrbublik/react-awesome-query-builder/security/code-scanning/10
To fix the issue, we need to ensure that backslashes are properly escaped in the default branch of the
escapeLike
function. This can be achieved by modifying the regular expression in theres.replace
call on line 40 to include backslashes (\\
) as part of the characters to escape. The replacement string should also escape backslashes correctly.The updated regular expression will be
/[%_\\]/g
, and the replacement string will be"\\\\$&"
. This ensures that all occurrences of%
,_
, and\
are escaped consistently.Suggested fixes powered by Copilot Autofix. Review carefully before merging.