jellyfin / jellyfin-plugin-opensubtitles

https://jellyfin.org
GNU General Public License v3.0
121 stars 26 forks source link

Replace %20 with + in SearchSubtitlesAsync #89

Closed MBR-0001 closed 2 years ago

MBR-0001 commented 2 years ago

per https://github.com/jellyfin/jellyfin-plugin-opensubtitles/issues/73#issuecomment-989088709

crobibero commented 2 years ago

I don't think we were supposed to change it on our end, this was something that I think OpenSubtitles fixed on their end

MBR-0001 commented 2 years ago

Well yes and no, atm the plugin works and everything but by replacing %20 with + redirection is avoided and the response is obtained faster

The new rule redirects /api/v1/subtitles?query=test%20test to /api/v1/subtitles?query=test+test (curl -v https://api.opensubtitles.com/api/v1/subtitles?query=test%20test to check it out)