fix invalid proxy_hide_header in example config #472

Merged
floatingghost merged 2 commits from flisk/akkoma:remote-media-docs-fix into develop 3 weeks ago
flisk commented 1 month ago

proxy_hide_header takes one header per statement in any nginx version i'm aware of, so the current version is probably a typo.

somewhat related: the media proxy config on this page differs a bit from the one in installation/nginx/akkoma.nginx, and it's not really obvious to me which one i should be using. i'm still looking into this, but maybe someone has input on how i could make the docs more consistent?

`proxy_hide_header` takes one header per statement in any nginx version i'm aware of, so the current version is probably a typo. somewhat related: the media proxy config on this page differs a bit from the one in `installation/nginx/akkoma.nginx`, and it's not really obvious to me which one i should be using. i'm still looking into this, but maybe someone has input on how i could make the docs more consistent?
flisk added 1 commit 1 month ago
ci/woodpecker/pr/woodpecker Pipeline is pending Details
107a72e04c
fix invalid proxy_hide_header in example config
Owner

hm curious

maybe there was an old nginx version that allowed this

the media proxy config on this page differs a bit from the one in installation/nginx/akkoma.nginx, and it's not really obvious to me which one i should be using

the one in installation should be correct, so if it could match that it'd be great

hm curious maybe there was an old nginx version that allowed this > the media proxy config on this page differs a bit from the one in installation/nginx/akkoma.nginx, and it's not really obvious to me which one i should be using the one in `installation` should be correct, so if it could match that it'd be great
flisk force-pushed remote-media-docs-fix from 107a72e04c to 3c2fa14634 4 weeks ago
flisk commented 4 weeks ago
Poster

how's this?

how's this?
flisk added 1 commit 4 weeks ago
ci/woodpecker/pr/woodpecker Pipeline is pending Details
fb159af483
thanks gitea
flisk added 1 commit 4 weeks ago
flisk force-pushed remote-media-docs-fix from a5fc20626c to 1ab0b3a0e2 4 weeks ago
flisk commented 4 weeks ago
Poster

(little bit of thrashing to make gitea render that link correctly, sorry)

(little bit of thrashing to make gitea render that link correctly, sorry)
Owner

that's good, thanks a lot!

that's good, thanks a lot!
floatingghost merged commit c8add9d1dc into develop 3 weeks ago
floatingghost deleted branch remote-media-docs-fix 3 weeks ago
The pull request has been merged as c8add9d1dc.
Sign in to join this conversation.
Loading…
There is no content yet.