Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unable to mark as read behind http to https reverse proxy redirect #292

Open
sudogoku opened this issue Feb 18, 2024 · 0 comments
Open

Unable to mark as read behind http to https reverse proxy redirect #292

sudogoku opened this issue Feb 18, 2024 · 0 comments

Comments

@sudogoku
Copy link

Describe the bug
When accessing the server via a reverse proxy, the Sorayomi app shows "308" as an error and the request is not completed & the chapters remain unread. This only happens when using a reverse proxy set up with a http to https redirect to access the server.
Obviously, "308" permanent redirect is being interpreted by Sorayomi as an error (I'm assuming it just checks for non-2xx status codes). After updating the server URL to explicitly include "https", this is no longer an issue.

To Reproduce
Steps to reproduce the behavior:

  1. Have a reverse proxy behind a domain name that serves traffic to Tachidesk server.
  2. Enter "http://my.domain.name" in the server URL settings
  3. Go to library, try to mark any chapter as read
  4. See "308" error message pop up

Expected behavior
Marking as read should still work as expected. Even though this is easily fixed by including "https" explicitly in the URL, a 308 redirect is pretty standard.

Screenshots
Probably not necessary, but will add if requested.

Device information (Server)

  • Tachidesk version: v0.7.0-r1487
  • Server Operating System: Ubuntu 22.04.3
  • Server Desktop Environment: N/A (headless server)
  • Server JVM version: N/A (using most recent preview release of suwayomi/tachidesk:preview docker image)

Device information (Client)

  • Client version: (Example: 0.1.1)
  • Client Type: iPhone
  • Client Operating System: iOS 17.2.1
  • Client Specifications: iPhone 13 Pro
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant