-
-
Notifications
You must be signed in to change notification settings - Fork 25
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
Download error: System.Xml.XmlException: Root element is missin #41
Comments
I am having problems with opensubtitles too. My logs are:
|
Wrong repository. Moved. |
Any updates on this? |
I am getting this on 10.7.0 |
I was getting this error. I uninstalled the plugin, restarted jellyfin, re-installed the plugin, and restarted jellyfin again and that seemed to take care of it. |
I can reproduce on I am logged in to a premium opensubtitle account, have a few plugins installed, and running 10.7.1.
Before updating to the
I will try the workaround posted above with reinstalling the plugin. |
Reinstalling the plugin and deleted config doesn't work. The only thing I can think of that would be specific to my configuration is that I have a premium Opensubtitles account. The error seems to come from that function, but I don't yet have a dev environment, so debugging is tricky:
|
I think OpenSubTitles changed their api responses, atleast thats what I can think of when errors like |
Yes, that wouldn't be surprising. BTW, would you mind changing this issue title so something a bit more descriptive like |
@voarsh2 fyi Open Subtitles 10 is the version you should use for 10.7. |
OpenSubtitles plugin 10, jellyfin 10.7.2, same errors
|
Does anyone know if this bug is going to be resolved soon? |
I guess I am not the only one. This a major plugin bug afaik |
Has anyone found a fix yet? |
No fix or response from the dev team it seems. |
Looks like the plugin still works for some people, though. I have a VIP account, I wonder if this is related. To whoever reads this, could you react with:
Thanks. This could help diagnosing the issue. |
I have changed my password to a very simple one (no ciphers or symbols) and now I get different errors: |
I'm having the same error. Jellyfin: v10.7.1 on Docker. Plugin version: 10.0.0 Glad to help if I can |
Well to me it still looks like an endpoint problem. Like OpenSubtitles changed the way the API requests response, I do not have time to look into it, but if someone else could try looking into their api and maybe look into the changes made since begin April maybe. |
If I build the plugin using the latest git version, I get this error instead after running the
I'm not sure if it's related. |
@Robpizza I tried looking for some kind of changelog but personally I find it really hard to find some kind of official source. Their NPM package was last updated over a year ago (don't know if that is official or not). However they upgrading their API as you can see here: https://trac.opensubtitles.org/projects/opensubtitles. Maybe that is causing this error? However I would also like to point out that these issues have started appearing with V10 of this plugin, this can be a coincidence but I doubt it to be honest. |
Latest git version is broken I've rewritten the plugin to use the new REST API, mostly for my personal use but if anyone wants I can give instructions on how to get it running (it uses the new version of opensubtitles, opensubtitles.com so you'd need to migrate your account). From what I've seen so far it works fine, definitely better than the 10.0.0.0 lol |
@MBR-0001 I tested your fork, but I get this error:
EDIT: Nevermind, it suddenly started working. |
Since you have it working, why not make a PR to bring your changes into the org? Every time I've tested this plugin it works fine for me, so it's very difficult to fix. |
Well I originally thought about it but then I found out the new API is in beta and it requires account migration, if that's not a problem I guess I could open a PR 🤔 |
Ah I see.. Glad you have it working, hopefully the new API will be ready for production use soon |
Well, this is just my opinion, but I would rather use a beta API that works (or partially works) than nothing at all. |
I asked the developer what the current state of new API is and got the following response
I'll open a PR if you think this is considered good enough |
I think a PR should be opened. Simply because as it stands the current implementation isn't working, so it might as well be considered down 24/7 for those of us experiencing this issue. As for their lack of monitoring, they can easily use a free service like UptimeRobot to monitor http endpoints or even better, a NetData server for in depth metrics and monitoring. |
@MBR-0001 I've built and ran the plugin and it works very nicely. However now I'm getting alot of Authentication to Opensubtitles failed errors while the username, password and api key are all correct. Any idea what might be causing this? |
Their API seems to be down, I believe the website doesn't use it which is why it works It'd probably be a good idea to include a link to the status page in the error message 🤔 EDIT: to keep this issue "clean" it would probably be best to do rewrite-related discussions in #54 |
Describe the bug
The plugin does not work in the latest version.
System (please complete the following information):
To Reproduce
Install the plugin, login and run the scheduled task.
Expected behavior
Subtitle download, but it throws errors.
Logs
Additional context
This is Anime content, I do not know if this is a direct problem!
The text was updated successfully, but these errors were encountered: