influxdata / influxdata/influxdb

Delete (or move) hand-written HTTP clients

Open
#21,801 0 comments 0 reactions 0 assignees View on GitHub
area/2.x area/HTTP kind/tech-debt
Dominant language
Rust
Stars
31.7k
Forks
3.7k
Avg merge
13h 37m
Merged PRs (30d)
8

Description

#21797 removed the CLI code from this repo, but left the hand-written HTTP clients because they're heavily used by integration tests.

- [ ] Determine which clients are safe to delete, and which are used by tests
- [ ] Delete all clients that are safe
- [ ] Consolidate clients used for testing into a single package focused on testing

Item 3 will probably require significant refactoring, because the existing clients use package-private constants for things like HTTP prefixes. We could possible adapt the codegen system used by the new CLI repo to generate HTTP clients here.

Contributor guide

Open the contributing guide

Research direction

Start by inventorying the hand-written HTTP clients and searching the integration tests for their use. Done means deleting clients that are safe to remove and consolidating test-used clients into one testing-focused package; the issue notes that this may require significant refactoring or code generation.

Written by the indexing model from the issue text.

Assessment

Domain
testing
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.