linuxserver / reverse-proxy-confs

These confs are pulled into our SWAG image: https://github.com/linuxserver/docker-swag
GNU General Public License v3.0
1.38k stars 310 forks source link

Fix calibre-web.subdomain.conf.sample to work with Kobo #714

Open Snuupy opened 3 weeks ago

Snuupy commented 3 weeks ago

linuxserver.io



Description

Current reverse proxy config doesn't work for kobo

I don't necessarily expect this PR to get merged cus I actually think it's ugly that the proxy_set_header stuff is all repeated but it is what it is (current default borks for kobo, this one doesn't)

please feel free to refactor as necessary so it's not dirty

Benefits of this PR and context

kobo sync works now instead of fail/error, downloads work too

How Has This Been Tested?

I have a kobo and this works, original conf doesn't sync NOR download

Source / References

https://github.com/janeczku/calibre-web/issues/1891#issuecomment-801886803

jgillman commented 2 weeks ago

Oh and you can also add the same changes to the calibre-web.subdomain.conf.sample version for completion :)

aptalca commented 3 days ago

Also, port 80 listen and http-->https redirect is not necessary as it's handled by default in default.conf