ga4gh / ga4gh/refget

refget: v2 spec for Range header errors does not align with typical usage

Open
#89 0 comments 0 reactions 0 assignees View on GitHub
invalid refget-sequence
Dominant language
No language data
Stars
17
Forks
10
Avg merge
11m
Merged PRs (30d)
1

Description

Creating on behalf of @davidlougheed as the GitHub issue transfer code wasn't working for me.

---

For range headers, it is [generally accepted](https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/416) that the `Range Not Satisfiable` error is to be used if a "document doesn't contain such ranges".

In Refget v2, `400 Bad Request` is used for this instead; I believe a 416 would be better because it can then require a [Content-Range](https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Content-Range) response header with the total sequence length, allowing clients to subsequently discover valid range queries that can be made and aligning better with the original intent of HTTP range responses.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.