-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
OPDS: Don't needlessly setup Basic auth #7372
Conversation
That causes us to try Basic auth, which is useless (and may or may not be problematic for some servers).
Some servers may still refuse to obey, though. They're breaking RFC2616 by doing so.
Maybe some workarounds especially for this server? |
PRs welcome to handle But that server is hella broken. (e.g., it does send a |
It already handles setting up the Host & Basic authentication headers
On that server I cannot even get the books list, OPDS browser gives a message 'Catalog not found'. |
Yep, because it's serving gzip data instead of plaintext ;). |
InputDialog returns empty strings, not nil. Explicitly nil username
& passwordwhen they're empty to avoid enabling authentication.Also, re #7370, explicitly request no compression. Doesn't fix the actual issue because the server doesn't obey, breaking the RFC. Most servers were already assuming no Accept-Encoding header meant that we really didn't want gzip data ;p.
This change is![Reviewable](https://camo.githubusercontent.com/23b05f5fb48215c989e92cc44cf6512512d083132bd3daf689867c8d9d386888/68747470733a2f2f72657669657761626c652e696f2f7265766965775f627574746f6e2e737667)