influxdata / influxdata/influxdb

InfluxDB 2: Deleting not exists label or label with not exists resource should produce `404 Not Found` response

Open
#18,409 1 comment 0 reactions 0 assignees View on GitHub
area/api team/compute
Dominant language
Rust
Stars
31.7k
Forks
3.7k
Avg merge
13h 37m
Merged PRs (30d)
8

Description

It's should works for all types of resource:

- Dashboards
- Notifications
- Checks
- TelegrafConfigs
- NotificationRules
- Buckets
- Variables
- Tasks
- ...

```bash
# Delete label for not exists resource
--> DELETE http://127.0.0.1:9999/api/v2/buckets/ffffffffffffffff/labels/ffffffffffffffff

<-- 204 No Content http://127.0.0.1:9999/api/v2/buckets/ffffffffffffffff/labels/ffffffffffffffff
```

__Expected behavior:__
Produce `404` HTTP response.

__Actual behavior:__
Produce `204 No Content` HTTP response.

__Environment info:__

* InfluxDB version: latest nightly build

cc @AlirieGray

Contributor guide

Open the contributing guide

Research direction

Start by reproducing the DELETE request shown in the issue, then trace the API handlers for labels across the listed resource types. Done means deletion of a missing label or missing resource consistently returns HTTP 404 rather than 204 for all applicable resources.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
api, database
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 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.