Use uppercase HTTP HEAD method for media preview proxy request #128

Merged
floatingghost merged 1 commit from :media-proxy-preview-http-head-method-case into develop 2022-07-30 21:58:15 +00:00

Currently media proxy for preview is using "head" HTTP method, which while in lowercase is not recognized by many web servers, and particularly not very liked by nginx, rendering media preview proxy non-functional in many configurations. This MR canonicalizes it to uppercase "HEAD".

Currently media proxy for preview is using `"head"` HTTP method, which while in lowercase is not recognized by many web servers, and particularly not very liked by nginx, rendering media preview proxy non-functional in many configurations. This MR canonicalizes it to uppercase `"HEAD"`.
Ghost force-pushed media-proxy-preview-http-head-method-case from 2c5b2675f8
Some checks failed
ci/woodpecker/pr/woodpecker Pipeline failed
to 38f765bd0a
All checks were successful
ci/woodpecker/pr/woodpecker Pipeline was successful
2022-07-30 20:52:42 +00:00
Compare

whilst i cannot deny the correctness of this PR, your bio is uh... questionable and seems in attempt to provoke

let's stick on the civil side of things, and we'll have absolutely no reason to quarrel over anything, eh? it's in both our interests to keep this maintained

but for now, thanks for the contribution

whilst i cannot deny the correctness of this PR, your bio is uh... questionable and seems in attempt to provoke let's stick on the civil side of things, and we'll have absolutely no reason to quarrel over anything, eh? it's in both our interests to keep this maintained but for now, thanks for the contribution
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 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
AkkomaGang/akkoma!128
No description provided.