Skip to content
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

Refactor of request params parsing process #947

Draft
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

numbata
Copy link
Contributor

@numbata numbata commented Feb 17, 2025

This pull request splits the request parameter handling into separate parser classes for headers, routes, and bodies. This means the code is more organized and easier to work with, making life easier for contributors and maintainers. It's a step toward adding new features-like contract-based parameter parsing-without having to tangle everything up in one place.

By separating concerns, developers can more easily customize or extend the way parameters are handled. In the future, we will be able to add stricter validation or additional checks simply by creating or updating a parser class. Overall, these changes should keep things cleaner, pave the way for stronger parameter validation, and help the project evolve smoothly.

In two words: my main goal here is to be able to add contract-based request params parser as one of the next steps.

@numbata numbata force-pushed the request-param-parsers branch from 719d29c to 55a94d0 Compare February 18, 2025 09:29
@numbata
Copy link
Contributor Author

numbata commented Feb 18, 2025

Some changes extracted into #948.

@numbata numbata force-pushed the request-param-parsers branch from a99e07a to 011800a Compare February 19, 2025 14:52
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