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

Define an extractor for all of a request's parameters #164

Open
jwodder opened this issue Jul 13, 2024 · 0 comments
Open

Define an extractor for all of a request's parameters #164

jwodder opened this issue Jul 13, 2024 · 0 comments
Assignees
Labels
code quality Nonfunctional improvements to the codebase high priority Work on these first

Comments

@jwodder
Copy link
Member

jwodder commented Jul 13, 2024

  • Define the following type:

    enum DavRequest {
        Get {path: DavPath, pathparts: Vec<Components>},
        Propfind {path: DavPath, depth: FiniteDepth, query: PropFind},
        Options,
    }
  • impl FromRequest<S> for DavRequest

  • Use this to replace the content of inner_handle_request() with a single match on a req.extract::<DavRequest, _>() call

    • If possible/reasonable, move all of inner_handle_request() into handle_request()
@jwodder jwodder added the code quality Nonfunctional improvements to the codebase label Jul 13, 2024
@jwodder jwodder added this to the Current priorities milestone Jul 13, 2024
@jwodder jwodder self-assigned this Jul 13, 2024
@jwodder jwodder added the high priority Work on these first label Jul 22, 2024
@jwodder jwodder removed this from the Current priorities milestone Jul 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code quality Nonfunctional improvements to the codebase high priority Work on these first
Projects
None yet
Development

No branches or pull requests

1 participant