search

linuxserver / docker-cops

GNU General Public License v3.0
40 stars 13 forks source link

Fix rewriting rules default site conf - see linuxserver/docker-cops#31 #52

Closed mikespub closed 6 months ago

mikespub commented 6 months ago

Description:

Fix rewriting rules in default site conf according to COPS guidelines - see https://github.com/seblucas/cops/wiki/Url-Rewriting-with-COPS

Benefits of this PR and context:

Fix downloading or viewing a book with any e-reader having difficulties fetching books from /fetch?php?data=1&type=epub links like Kobo, Moon+ Reader etc.

Closes issue #31 and replaces PR #33

How Has This Been Tested?

Enable url rewriting in COPS config_local.php file

$config['cops_use_url_rewriting'] = "1";

Source / References:

See https://github.com/seblucas/cops/wiki/Url-Rewriting-with-COPS

LinuxServer-CI commented 6 months ago
I am a bot, here are the test results for this PR: https://ci-tests.linuxserver.io/lspipepr/cops/2.5.6-pkg-4e338e49-dev-efe5296e70fc25325e7ad843c1798ce0c8bb1a3f-pr-52/index.html https://ci-tests.linuxserver.io/lspipepr/cops/2.5.6-pkg-4e338e49-dev-efe5296e70fc25325e7ad843c1798ce0c8bb1a3f-pr-52/shellcheck-result.xml Tag Passed
amd64-2.5.6-pkg-4e338e49-dev-efe5296e70fc25325e7ad843c1798ce0c8bb1a3f-pr-52 ✅
arm64v8-2.5.6-pkg-4e338e49-dev-efe5296e70fc25325e7ad843c1798ce0c8bb1a3f-pr-52 ✅
aptalca commented 6 months ago

Thanks for the PR 👍