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

Failed to send notification using default #95

Closed
SteveEdson opened this issue Aug 31, 2016 · 5 comments
Closed

Failed to send notification using default #95

SteveEdson opened this issue Aug 31, 2016 · 5 comments

Comments

@SteveEdson
Copy link

$ ntfy send 'test'

ERROR: Failed to send notification using default
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/site-packages/ntfy/__init__.py", line 48, in notify
    **backend_config)
  File "/usr/local/lib/python2.7/site-packages/ntfy/backends/default.py", line 22, in notify
    raise DefaultNotifierError(e, module)
DefaultNotifierError

However using ntfy -b pushover -o user_key xxx send 'Pushover test!' works fine

My config looks like

$ cat ~/.config/ntfy/ntfy.yml

---
backends:
    - pushover
pushover:
    user_key: xxx

Mac OS X 10.11.6 Beta

@SteveEdson SteveEdson changed the title Failed to send Failed to send notification using default Aug 31, 2016
@SteveEdson
Copy link
Author

SteveEdson commented Aug 31, 2016

Oh, using verbose debug output I get this:

INFO: /Users/Steve/Library/Application Support/ntfy/ntfy.yml not found

Any reason this would be looking here? And it seems like that issue should be more prominent, maybe part of the exception message?

@dschep
Copy link
Owner

dschep commented Aug 31, 2016

  1. That INFO message is right, you can use that location(on MaOS) or ~/.ntfy.yml. The ~/.config location is used on linux. I'll update the README to use the more platform-agnostic ~/.ntfy.yml.
  2. Since it isn't finding the config, it's trying to send a desktop notification which per did it support mac? #87 the current version doesn't work unless you pip install pyobjc.

@SteveEdson
Copy link
Author

Updating the readme would be great thanks.

@korvus81
Copy link

korvus81 commented Jan 8, 2017

Also need to fix this section: https://github.com/dschep/ntfy/blob/master/README.rst#configuring-ntfy

Really confused me until I found this issue.

@dschep
Copy link
Owner

dschep commented Jan 8, 2017

Thanks for letting me know @korvus81. I've updated the docs so it should be clearer now 👍

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

3 participants