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

If I only have plex, the app errors out that I didn't set my Jellyfin URL #6

Open
joshkerr opened this issue Dec 4, 2024 · 14 comments

Comments

@joshkerr
Copy link

joshkerr commented Dec 4, 2024

It should skip Jelly and look for plex.

@Nairooolf
Copy link

same here

@THEGOLDENPRO
Copy link
Member

@r3tr0ananas could you possibly have a look at this, I think this is completely stopping the use of the plex scraper

@r3tr0ananas
Copy link
Member

can't reproduce this issue.

@THEGOLDENPRO
Copy link
Member

With your jellyfin environment variables unset too?

I just have a suspicion with that this line is causing it because as far as I remember scraper's init functions are executed no matter if you're using that scraper or not and self.__auth just happen to raise an exception if envs are missing.

self.new_headers, self.user_id, self.api_key = self.__get_auth()

sorry that I'm asking you to look into it, I'm just really busy

@r3tr0ananas
Copy link
Member

yep, my jellyfin env is unset and iirc init functions only are executed when you use that scraper.

@r3tr0ananas
Copy link
Member

@joshkerr what command do you use?

@rogers1lt
Copy link

I have just started using this plugin this afternoon with the similar error when using the command mov-cli -s jelly "query":
Error: You haven't set the 'JELLY_URL' env!

@r3tr0ananas
Copy link
Member

You need to do: mov-cli -s jelly.plex "query"

@rogers1lt
Copy link

That resolved my issue. I would request updating the documentation: Usage

@THEGOLDENPRO
Copy link
Member

THEGOLDENPRO commented Dec 8, 2024

or you can also override the jelly namespace using the config like this:

[mov-cli.scrapers]
jelly = "jellyplex.plex"

[mov-cli.plugins]
jellyplex = "mov-cli-jellyplex"

more on that in the wiki: https://github.com/mov-cli/mov-cli/wiki/Configuration#namespace-overriding

now -s jelly should point to the plex scraper and you can use this command like normal, but it's up to you:

mov-cli -s jelly "query"

@joshkerr
Copy link
Author

joshkerr commented Dec 9, 2024

Did not resolve my issue. Its still erroring out for me. I checked that my plex environment variables are set and I do have my mov-cli config loading the plugin into the jelly namespace, but I'm still getting this output:

mov-cli -s jelly.plex ghost

Good Evening, joshkerr.
It's 08:54 PM on a gorgeous Sunday!

[INFO] (mov_cli) - Using 'jelly.plex' scraper...
[INFO] (mov_cli) - Searching for 'ghost'...
[INFO] (mov_cli) - [ATNS] ❌ Error occurred while searching with 'jelly.plex'! Trying the next scraper...

@rogers1lt
Copy link

rogers1lt commented Dec 9, 2024 via email

@joshkerr
Copy link
Author

joshkerr commented Dec 9, 2024

Actually, that did fix my issue. I was working in a terminal that didn't have my env variables set because it wasn't refreshed. Once I did that, it worked fine.

@joshkerr
Copy link
Author

joshkerr commented Dec 9, 2024

Try turning debug=true in your settings for mov-cli and it will show you why its erroring out. That's how I figured it out.

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

5 participants