search

knightcrawler-stremio / knightcrawler

A selfhosted Stremio addon
Apache License 2.0
238 stars 39 forks source link

Issues with Premiumize #93

Open asylumexp opened 5 months ago

asylumexp commented 5 months ago

Describe the bug Playing media with stremio through premiumize is mostly broken: image

For probably 95% of media this error screen appears.

In the logs it either will say: Failed Premiumize adding torrent "No recent torrent found" or Failed Premiumize adding torrent {}

To Reproduce Steps to reproduce the behavior: Install Knightcrawler stremio addon with a premiumize account linked. Play any media. Almost certain chance it will not work.

Expected behavior To play the media or say the media is being downloaded.

Logs logs on pastebin

Hardware:

Additional context The logs aren't detailed because it isn't particularly clear anywhere to how to change the log level. The files are actually being added to my premiumize instance, and can be played from the 'View Content' button on the banner that shows up after adding an addon. This isn't an issue when played through the original torrentio. Out of probably 40 to 60 files i tried playing I only had 3 actually work, and couldn't really see any pattern with them. I had this happen on both the public and my hosted instance of knightcrawler.

iPromKnight commented 4 months ago

I don't have a premiumize account, so am unable to tests sorry - but I think its just a case of bringing in the upstream moch like Gabi has done in #97

purple-emily commented 4 months ago

@asylumexp can you test this fix? Are you able to pull and build the branch and try it?

asylumexp commented 4 months ago

@asylumexp can you test this fix? Are you able to pull and build the branch and try it?

I'll have a try of it

asylumexp commented 4 months ago

@asylumexp can you test this fix? Are you able to pull and build the branch and try it?

I'll have a try of it

ive been trying for a bit but i keep getting an error saying my premiumize key is incorrect image ive reinstalled the addon several times and its definitely the right API key

purple-emily commented 4 months ago

@asylumexp this fix won’t be live on ElfHosted. It would need to be locally built.

@Gabisonfire you want to merge the PR and see?

Gabisonfire commented 4 months ago

Yeah I guess it doesn't hurt anyways, I've just copied from the upstream anyways.

asylumexp commented 4 months ago

@asylumexp this fix won’t be live on ElfHosted. It would need to be locally built.

@Gabisonfire you want to merge the PR and see?

I talked to @funkypenguin and apparently should already have it according to him. image Perhaps im misunderstanding something but I thought that would mean the patch was on my instance

Gabisonfire commented 4 months ago

He pushed the update about 2 hours ago,

funkypenguin commented 4 months ago

Usually ElfHosted apps get updated during a daily release/maintenance cycle, but alpha-channel users get updates as they're available. So yes, @asylumexp should have the latest code in his hosted instance, and it's also on https://torrentio.elfhosted.com / https://knightcrawler.elfhosted.com

Gabisonfire commented 4 months ago

@asylumexp It this still an issue?