We should return 204 instead of 200 when returning an empty body #376

New Issue

Closed

opened 2022-09-12 09:27:14 +00:00 by quentin · 0 comments

quentin commented

2022-09-12 09:27:14 +00:00

Owner

Currently, we return StatusCode::OK (200) -> https://docs.rs/hyper/latest/hyper/struct.StatusCode.html#associatedconstant.OK

We should return instead StatusCode::NO_CONTENT (204) -> https://docs.rs/hyper/latest/hyper/struct.StatusCode.html#associatedconstant.NO_CONTENT

Currently, we return `StatusCode::OK` (200) -> https://docs.rs/hyper/latest/hyper/struct.StatusCode.html#associatedconstant.OK We should return instead `StatusCode::NO_CONTENT` (204) -> https://docs.rs/hyper/latest/hyper/struct.StatusCode.html#associatedconstant.NO_CONTENT

quentin added the

labels 2022-09-12 09:27:14 +00:00

quentin added this to the v0.9 milestone 2022-09-14 13:26:32 +00:00

tobikris referenced this issue

2022-10-16 22:42:58 +00:00

Use status code 204 No Content for empty responses #403

lx referenced this issue

2022-10-17 07:37:31 +00:00

Use status code 204 No Content for empty responses #403

lx referenced a pull request that will close this issue

2022-10-18 14:20:41 +00:00

Use status code 204 No Content for empty responses #403

lx closed this issue

2022-10-18 14:20:45 +00:00

Sign in to join this conversation.

No Label

S3 Compatibility

No Milestone

No Assignees

1 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Deuxfleurs/garage#376

No description provided.