Voeg Quarkus voorbeeld toe by TimvdLippe · Pull Request #1 · Logius-standaarden/API-mod-transfer

TimvdLippe · 2026-08-13T13:45:21Z

No description provided.

github-actions · 2026-08-13T13:46:39Z

Preview link voor pull request
Diff met W3CDiff

TimvdLippe · 2026-08-19T12:37:46Z

Tests in examples/quarkus/src/test/java/org/acme/ServerToClientResourceTest.java laten zien dat alle headers correct worden gezet

sanderke · 2026-09-08T16:33:11Z

+        int endIndex = rangeHeaderIndices.endIndex();
+
+        var totalFileSize = fullFileContent.length();
+        if (startIndex > totalFileSize || endIndex > totalFileSize) {


Suggested change

if (startIndex > totalFileSize || endIndex > totalFileSize) {

if (startIndex >= totalFileSize) {

Mocht je enkel de laatste byte willen, dan zijn ze gelijk. En endIndex te groot mag ook:

If the last-pos value is absent, or if the value is greater than or equal to the current length of the representation data, the byte range is interpreted as the remainder of the representation (i.e., the server replaces the value of last-pos with a value that is one less than the current length of the selected representation).

sanderke · 2026-09-09T14:07:23Z

+                    .withDetail("End index should be larger than start")
+                    .build();
+        }
+        return new RangeHeaderIndices(startIndex, endIndex);


Suggested change

return new RangeHeaderIndices(startIndex, endIndex);

return new RangeHeaderIndices(startIndex, endIndex+1);

Of totalFileSize als eindpunt, als dat eerder komt.

sanderke · 2026-09-09T14:41:39Z

+      assert.equal(headers.get("Content-Digest"), "sha-256=:4BVa2bszMXotZewPuxyxoNDP/ZAnBgNgdTRYjWn2uhY=:");
+      assert.equal(headers.get("Content-Range"), "bytes 10-12/35");
+      assert.equal(headers.get("ETag"), ETAG_FOR_FILE);
+      assert.equal(headers.get("Repr-Digest"), REPR_DIGEST_FOR_FILE);
+      assert.equal(fileResponse.status, 206);
+      assert.equal(await fileResponse.text(), "fil");


Suggested change

assert.equal(headers.get("Content-Digest"), "sha-256=:4BVa2bszMXotZewPuxyxoNDP/ZAnBgNgdTRYjWn2uhY=:");

assert.equal(headers.get("Content-Range"), "bytes 10-12/35");

assert.equal(headers.get("ETag"), ETAG_FOR_FILE);

assert.equal(headers.get("Repr-Digest"), REPR_DIGEST_FOR_FILE);

assert.equal(fileResponse.status, 206);

assert.equal(await fileResponse.text(), "fil");

assert.equal(headers.get("Content-Digest"), "sha-256=:O5w1jzbwoxtq0+FPMJx88ZiskkboMW+c5UPVsZrAK4A=:");

assert.equal(headers.get("Content-Range"), "bytes 10-13/35");

assert.equal(headers.get("ETag"), ETAG_FOR_FILE);

assert.equal(headers.get("Repr-Digest"), REPR_DIGEST_FOR_FILE);

assert.equal(fileResponse.status, 206);

assert.equal(await fileResponse.text(), "file");

Ah, zero-indexed & inclusive. Ja klopt

sanderke · 2026-09-09T14:42:57Z

+        assert.equal(headers.get("Content-Digest"), "sha-256=:4BVa2bszMXotZewPuxyxoNDP/ZAnBgNgdTRYjWn2uhY=:");
+        assert.equal(headers.get("Content-Range"), "bytes 10-12/35");
+        assert.equal(headers.get("ETag"), ETAG_FOR_FILE);
+        assert.equal(headers.get("Repr-Digest"), REPR_DIGEST_FOR_FILE);
+        assert.equal(fileResponse.status, 206);
+        assert.equal(await fileResponse.text(), "fil");


Suggested change

assert.equal(headers.get("Content-Digest"), "sha-256=:4BVa2bszMXotZewPuxyxoNDP/ZAnBgNgdTRYjWn2uhY=:");

assert.equal(headers.get("Content-Range"), "bytes 10-12/35");

assert.equal(headers.get("ETag"), ETAG_FOR_FILE);

assert.equal(headers.get("Repr-Digest"), REPR_DIGEST_FOR_FILE);

assert.equal(fileResponse.status, 206);

assert.equal(await fileResponse.text(), "fil");

assert.equal(headers.get("Content-Digest"), "sha-256=:O5w1jzbwoxtq0+FPMJx88ZiskkboMW+c5UPVsZrAK4A=:");

assert.equal(headers.get("Content-Range"), "bytes 10-13/35");

assert.equal(headers.get("ETag"), ETAG_FOR_FILE);

assert.equal(headers.get("Repr-Digest"), REPR_DIGEST_FOR_FILE);

assert.equal(fileResponse.status, 206);

assert.equal(await fileResponse.text(), "file");

sanderke · 2026-09-09T14:50:07Z

+    void testNegativeRangeTooLargeResultsInBadRequest() {
+        given().header("Range", "bytes=-130")
+                .when()
+                .get(LARGE_FILE_CONTENT_LOCATION)
+                .then()
+                .statusCode(416)
+                .contentType("application/problem+json")
+                .body("status", Matchers.equalTo(416))
+                .body("title", Matchers.equalTo("Invalid range header content"))
+                .body("detail", Matchers.equalTo("Negative range larger than file content"))
+                .body("instance", Matchers.equalTo(LARGE_FILE_CONTENT_LOCATION));
+    }


If the selected representation is shorter than the specified suffix-length, the entire representation is used.

sanderke · 2026-09-09T15:11:16Z

+    assert.equal(fileContent, LARGE_FILE_CONTENT);
+  });
+
+  describe("rejects retrieval with range", () => {


A server MUST ignore a Range header field received with a request method that is unrecognized or for which range handling is not defined.

Dus deze gevallen zouden dan 200 zijn met het hele bestand?

Ik interpreteerde die zin als "als je een Range doet met een POST, negeer die dan". Sowieso ben ik ook niet happy met de 200 hier heb je het hele bestand. Dat vind ik ook tegen intuitief voor een module die grote berichten in stukken wil verdelen.

Daarom zou ik dit strikt houden en 400 terug geven. Wat denk jij?

Ah, die ging over de method, maar de zin daarna lijkt wel echt over een voor de server onbegrijpelijke Range header field zelf te gaan:

An origin server MUST ignore a Range header field that contains a range unit it does not understand.

API-mod-transfer/examples/quarkus/src/test/java/org/acme/ServerToClientResourceTest.java

Lines 117 to 118 in 4930db2

void testRangeDoesNotStartWithBytesResultsInBadRequest() {

given().header("Range", "string=10-13")

Een bij de server (nog) niet geimplementeerde eenheid is nog niet meteen een client error, dus dan maar wel 200 "ik weet niet welk deel je wilt, dus hier is alles" (graceful?). De andere cases zijn zonder twijfel fout en dus inderdaad beter 400.

TimvdLippe

Thanks voor de review! Ik zal die punten verwerken. 1 comment met de interpretatie van die zin

TimvdLippe · 2026-09-10T10:54:23Z

+    assert.equal(fileContent, LARGE_FILE_CONTENT);
+  });
+
+  describe("rejects retrieval with range", () => {


Ik interpreteerde die zin als "als je een Range doet met een POST, negeer die dan". Sowieso ben ik ook niet happy met de 200 hier heb je het hele bestand. Dat vind ik ook tegen intuitief voor een module die grote berichten in stukken wil verdelen.

Daarom zou ik dit strikt houden en 400 terug geven. Wat denk jij?

TimvdLippe · 2026-09-10T10:55:02Z

+      assert.equal(headers.get("Content-Digest"), "sha-256=:4BVa2bszMXotZewPuxyxoNDP/ZAnBgNgdTRYjWn2uhY=:");
+      assert.equal(headers.get("Content-Range"), "bytes 10-12/35");
+      assert.equal(headers.get("ETag"), ETAG_FOR_FILE);
+      assert.equal(headers.get("Repr-Digest"), REPR_DIGEST_FOR_FILE);
+      assert.equal(fileResponse.status, 206);
+      assert.equal(await fileResponse.text(), "fil");


Ah, zero-indexed & inclusive. Ja klopt

Voeg Quarkus voorbeeld toe

8bb36eb

TimvdLippe added 3 commits August 19, 2026 13:51

Support alle headers van de standaard

a94e057

Voeg testsuite toe voor alle scenario's

33ba1f9

Return problem+json als bestand niet bestaat

2e6af23

TimvdLippe requested a review from sanderke August 19, 2026 12:37

TimvdLippe marked this pull request as ready for review August 19, 2026 12:37

TimvdLippe added 6 commits August 19, 2026 15:15

Support negative indices voor laatste bytes van een file

abf1d1b

Voeg client to server upload toe

4aa07c4

Fix error handling met uploaden vanaf client naar server

c468138

Fix issues die we vonden bij de review

e98df2e

Voeg JavaScript test suite toe

810eff7

Maak het mogelijk om een server URL te specificeren

4930db2

sanderke reviewed Sep 9, 2026

View reviewed changes

TimvdLippe commented Sep 10, 2026

View reviewed changes

	if (startIndex > totalFileSize \|\| endIndex > totalFileSize) {
	if (startIndex >= totalFileSize) {

	return new RangeHeaderIndices(startIndex, endIndex);
	return new RangeHeaderIndices(startIndex, endIndex+1);

	void testRangeDoesNotStartWithBytesResultsInBadRequest() {
	given().header("Range", "string=10-13")

Uh oh!

Conversation

TimvdLippe commented Aug 13, 2026

Uh oh!

github-actions Bot commented Aug 13, 2026

Uh oh!

TimvdLippe commented Aug 19, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

TimvdLippe left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants