influxdata / influxdata/influxdb

Revise update of scraper targets

Open
#10,797 5 comments 0 reactions 0 assignees View on GitHub
area/api
Dominant language
Rust
Stars
31.7k
Forks
3.7k
Avg merge
13h 37m
Merged PRs (30d)
8

Description

Currently the patch/update process for scraper targets does not use any struct to specify which fields can be updated, but uses the `platform.ScraperTarget` directly.

Thus I propose to introduce a `platform.ScraperTargetUpdate` struct and use another private struct to decode into it during `PATCH`.

Contributor guide

Open the contributing guide

Research direction

Start by locating the PATCH handler for scraper targets and the definition of platform.ScraperTarget. Review how the request body is decoded and which fields are currently accepted. Done means introducing the proposed update struct and private decode struct so PATCH updates only the intended fields.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
backend-api-design
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.