Skip to content

Prevent Identifier breakout via quoteSql() sequences - #1207

Merged
vjik merged 3 commits into
masterfrom
improve-identifier
Oct 18, 2026
Merged

vjik merged 3 commits into
masterfrom
improve-identifier

Conversation

@vjik

@vjik vjik commented Oct 14, 2026 •

Copy link
Copy Markdown
Member

@codecov

codecov Bot commented Oct 14, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.73%. Comparing base (6c56b69) to head (7d1c202).

Additional details and impacted files
@@            Coverage Diff            @@
##             master    #1207   +/-   ##
=========================================
  Coverage     98.73%   98.73%           
- Complexity     1663     1664    +1     
=========================================
  Files           125      125           
  Lines          4342     4350    +8     
=========================================
+ Hits           4287     4295    +8     
  Misses           55       55           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@vjik
vjik requested a review from a team October 14, 2026 08:34
@vjik vjik added the status:code review The pull request needs review. label Oct 14, 2026
Comment thread src/Expression/Value/Builder/IdentifierBuilder.php
@vjik
vjik merged commit 7855801 into master Oct 18, 2026
84 checks passed
@vjik
vjik deleted the improve-identifier branch October 18, 2026 07:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status:code review The pull request needs review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants