Skip to content

feat(core): Add ValidateQuery hook - #1368

Open
kingsthwaiteJ wants to merge 4 commits into
FoalTS:masterfrom
kingsthwaiteJ:create_validate_query_hook
Open

feat(core): Add ValidateQuery hook#1368
kingsthwaiteJ wants to merge 4 commits into
FoalTS:masterfrom
kingsthwaiteJ:create_validate_query_hook

Conversation

@kingsthwaiteJ

@kingsthwaiteJ kingsthwaiteJ commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Problem

Leverages the schema validation approach from ValidateBody and applies it in a new ValidateQuery hook that targets request.query.

Addresses #1367.

Solution

Refactor the ValidateBody hook and pull out the schema validation into a shared helper. Leverage that helper in the ValidateBody hook to preserve existing functionality.

Create a new ValidateQuery hook that leverages the same helper, add automated tests and update documentation.

Note: have made a first-pass at updating the docs but I'm aware that there may be gaps. Will wait for a maintainer to confirm.

Checklist

  • I have read the contribution guidelines.
  • I have added or updated tests for any code changes.
  • I have updated documentation if needed, keeping it concise.

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