Skip to content

Make clause security check pluggable#1441

Closed
qqxhb wants to merge 1 commit intomasterfrom
fix/issue-1115-unique-aware
Closed

Make clause security check pluggable#1441
qqxhb wants to merge 1 commit intomasterfrom
fix/issue-1115-unique-aware

Conversation

@qqxhb
Copy link
Member

@qqxhb qqxhb commented Feb 28, 2026

Summary

  • add ClauseChecker hook with ErrClauseNotHandled fallback behavior
  • centralize clause validation with optional custom checker
  • add tests for custom checker behavior

Tests

  • go test ./...

@propel-code-bot
Copy link

It also wires the configurable checker into DO.Clauses and centralizes fallback handling in a helper to preserve the existing default behavior unless overridden.

Affected Areas

do_options.go
sec_check.go
do.go
do_clause_checker_test.go

This summary was automatically generated by @propel-code-bot

Copy link

@propel-code-bot propel-code-bot bot left a comment

Choose a reason for hiding this comment

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

LGTM, ship it! 🚢

Details
Why was this auto-approved?
APPROVED: No issues detected.

@qqxhb
Copy link
Member Author

qqxhb commented Feb 28, 2026

--delete-branch

@qqxhb qqxhb closed this Feb 28, 2026
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.

1 participant