influxdata / influxdata/influxdb

Consolidate error handling in HTTP

Open
#17,889 1 comment 1 reaction 0 assignees View on GitHub
Dominant language
Rust
Stars
31.7k
Forks
3.7k
Avg merge
13h 37m
Merged PRs (30d)
8

Description

Method 1:
https://github.com/influxdata/influxdb/blob/feature/dbrp-api/kit/transport/http/api.go#L201

Method 2:
https://github.com/influxdata/influxdb/blob/feature/dbrp-api/kit/transport/http/error_handler.go#L18

We should really have one way to go.

Contributor guide

Open the contributing guide

Research direction

Start by comparing the HTTP handling in kit/transport/http/api.go around line 201 with kit/transport/http/error_handler.go from the linked branch. Trace how each method reports errors and decide what a single handling path should cover. Done means the two approaches are consolidated into one consistent mechanism.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
api, backend
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
28/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.