ankitpokhrel / ankitpokhrel/tus-php

Client implementation overly restrictive on HEAD requests

Offen
#455 0 Kommentare 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
PHP
Sterne
1.5k
Forks
220
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

> **Note:** Please report any security issues directly to hello [at] ankit.pl

**Describe the bug**
`sendHeadRequest` implementation in the Client only expects a 200-level status. The [spec](https://tus.io/protocols/resumable-upload#head) says a server SHOULD return 200 or 204, so the client should probably at least allow 204 statuses.

**To Reproduce**
Use the client to connect to a Tus server that returns a 204 status code on HEAD requests.

**Expected behavior**
Head request is treated according to the specification and uploading continues as expected.

I've prepared #454 as a fix.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.