Azure / Azure/data-api-builder

RestService Code Cleanup

Open
#749 0 comments 0 reactions 0 assignees View on GitHub
low priority refactor
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.