Azure / Azure/data-api-builder
RestService Code Cleanup
- Dominant language
- C#
- Stars
- 1.5k
- Forks
- 370
- Avg merge
- 3d 22h
- Merged PRs (30d)
- 9
Description
RestService would benefit from a refactor that cleans up the code and clearly defines methods in RequestParser/RequestValidator to adhere to the purpose defined by the class name: i.e. request parsing occurs in some methods in request validator and requestparser only parses URL content, not the full request.
Some work was done in #727 to begin cleanup.
Contributor guide
Research direction
Review RestService, RequestParser, and RequestValidator, then compare their current responsibilities with the cleanup started in issue #727. Trace which methods perform request parsing versus URL parsing and define completion as each class adhering to the purpose described in this issue.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- csharp
- Domain
- api
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100