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

[SABnzbd] ModuleNotFoundError Using discord:// destination in SABnzbd #68

Open
ceyounger opened this issue Sep 3, 2020 · 8 comments
Open
Labels

Comments

@ceyounger
Copy link

🪲 Describe the bug
Updated to the latest Notify.py and sabnzbd-notify.py scripts and getting error thrown in SABnzbd and from CLI when testing.

💡 Screenshots and Logs

# python sabnzbd-notify.py complete 'Hello' 'World' discord://XXXXXXX@XXXXXXXXXXXX/XXXXXXXXXXXXXXXXXXXXXXXXXXXX
Traceback (most recent call last):
  File "/config/scripts/Notify.py", line 174, in <module>
    from nzbget import SCRIPT_MODE
ModuleNotFoundError: No module named 'nzbget' 

💻 Your System Details:

  • OS: Docker (linuxserver/sabnzbd:latest)

🔮 Additional context
SABnzbd was throwing errors with the old script as I deleted a Discord Webhook. Took the opportunity to update the script and discovered this error. I'm no coder, but it looks like the check for if we are talking to NZBGet or SABnzbd is happening after module import. But, again, I'm talking out my a$$ here.

@ceyounger ceyounger added the bug label Sep 3, 2020
@ceyounger
Copy link
Author

running pip install pynzbget inside the container solves this issue.

Docker images need to be updated.

Closing this issue as it's not a problem with the script.

@caronc
Copy link
Owner

caronc commented Sep 10, 2023

I can tell you that it is not a bug with this script which hasn't seen an update in years. Likely the Docker container yours using has been recently rebuilt and there is an issue with the python libraries within it.

I do know that env isn't used as much anymore. One thing i could do is update the first line of the script to be #!/usr/bin/python3 which may resolve your issue (but the container you use would have to be rebuilt again).

Not sure if there are still Python 2 users, this would definitely break for them if I did this.

@caronc caronc reopened this Sep 10, 2023
@caronc
Copy link
Owner

caronc commented Sep 10, 2023

Change it to #!/usr/bin/python3 (no env reference)

@caronc
Copy link
Owner

caronc commented Sep 16, 2023

The problem isn't with the script; you proved this by changing the header of it before I did and it didn't help. There is an issue with the container your pulling down (or some change that has been applied to it afterwards).

I'm not quite sure what you mean by saying "use notifiarr for this", but if another tool solves your issue, that's actually great news! 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants
@caronc @ceyounger and others