fix invalid proxy_hide_header in example config #472

Merged

floatingghost merged 2 commits from flisk/akkoma:remote-media-docs-fix into develop

2023-03-02 11:19:47 +00:00

flisk commented

2023-02-18 13:17:02 +00:00

Contributor

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?

floatingghost commented

2023-02-26 22:03:04 +00:00

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

ci/woodpecker/pr/woodpecker Pipeline is pending

Details

to 3c2fa14634

ci/woodpecker/pr/woodpecker Pipeline is pending

Details

2023-02-26 22:39:49 +00:00

Compare

flisk commented

2023-02-26 22:40:24 +00:00

Author

Contributor