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

Ghost commented

2022-07-30 17:12:34 +00:00

First-time contributor

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 to 38f765bd0a

2022-07-30 20:52:42 +00:00

Compare

floatingghost commented

2022-07-30 21:58:04 +00:00

Owner

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