caronc / apprise

Apprise - Push Notifications that work with just about every platform!
https://hub.docker.com/r/caronc/apprise
BSD 2-Clause "Simplified" License
10.91k stars 384 forks source link

Custom Plugins fail to load due to deadlock #1075

Closed dtrodrigues closed 3 months ago

dtrodrigues commented 4 months ago

:mega: Notification Service(s) Impacted Custom Plugins

:lady_beetle: Describe the bug Custom Plugins fail to load due to deadlock

:bulb: Screenshots and Logs

:computer: Your System Details:

:crystal_ball: Additional context I put the following into ~/brewtest_notifier.py:

from apprise.decorators import notify

@notify(on="brewtest")
def my_wrapper(body, title, *args, **kwargs):
  print(f"{title}: {body}")

The following works with apprise 1.7.2:

$ apprise -P ~/brewtest_notifier.py -t mytitle -b mybody brewtest://
mytitle: mybody

but it hangs for when using anything after https://github.com/caronc/apprise/pull/1071 was merged. It can also be seen in the CI timeouts in https://github.com/Homebrew/homebrew-core/pull/164950 .

caronc commented 3 months ago

Closing issue as resolved (test cases added to cover from ever happening again in the future)