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

[FEAT] Include paho-mqtt to enable Apprise mqtt:// support #146

Open
1 task done
brycied00d opened this issue Nov 22, 2024 · 1 comment
Open
1 task done

[FEAT] Include paho-mqtt to enable Apprise mqtt:// support #146

brycied00d opened this issue Nov 22, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@brycied00d
Copy link

Is this a new feature request?

  • I have searched the existing issues

Wanted change

I would like for the paho-mqtt Python module to be included in this image.

Reason for change

It would be helpful to me if Bazarr's Apprise feature would support mqtt:// URIs. The linuxserver/docker-sabnzbd image already supports this (https://github.com/linuxserver/docker-sabnzbd/blame/4.3.0-ls160/package_versions.txt#L80)

At present, attempting to configure a notification in Bazarr that uses "Apprise API" with an "mqtt://" URI results in an error:

2024-11-22 13:50:09,830 - apprise                          (790b3cc1cb30) :  ERROR (Apprise:173) - mqtt:// is disabled on this system.
2024-11-22 13:50:09,830 - apprise                          (790b3cc1cb30) :  ERROR (Apprise:456) - There are no service(s) to notify

That error is the result of bazarr attempting and failing to import the paho-mqtt module (https://github.com/morpheus65535/bazarr/blob/v1.4.5/libs/apprise/plugins/NotifyMQTT.py#L48-L56)

Proposed code change

I'm not too familiar with LSio's image building system so I'm afraid I don't know how to add this. It appears that with linuxserver/docker-sabnzbd, it was automatically pulled in by sabnzbd's own requirements.txt with their introduction of the Apprise integration (sabnzbd/sabnzbd#2796). Does this issue needs to be taken upstream to bazarr for them to update their requirements.txt?

@brycied00d brycied00d added the enhancement New feature or request label Nov 22, 2024
Copy link

Thanks for opening your first issue here! Be sure to follow the relevant issue templates, or risk having this issue marked as invalid.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Issues
Development

No branches or pull requests

1 participant