apprise: Error "Failed to send to Slack: error=200", but notification arrives
📣 Notification Service(s) Impacted Slack
🪲 Describe the bug
When using the Apprise CLI v0.8.3 to notify a Slack channel with the method “incoming web hook”, apprise
exits with exit code 1, but the notification is delivered.
When using verbose mode -vvv
, the following lines shows up in the log:
2020-01-17 22:42:59,617 - WARNING - Failed to send to Slack: error=200.
2020-01-17 22:42:59,617 - DEBUG - Response Details:
ok
The Slack-URL I’m using has the following format:
slack://API-Deployer@xxx/xxx/xxx/#channelname
đź’» Your System Details: Tested on two systems:
- OS: MacOS 10.15.2 (Catalina) and Ubuntu-latest
- Python 2.7.16 (MacOS) and 3.7 (Ubuntu)
đź”® Additional context
- Not tested with the bot method.
- This problem didn’t show up in version 0.7.8, the previous one I had.
- It shows up in my logs starting end of November, might have been introduced in 0.8.2.
- Tested also the Pushover service, this works flawlessly on both machines.
About this issue
- Original URL
- State: closed
- Created 4 years ago
- Comments: 28 (20 by maintainers)
Commits related to this issue
- resolves slack error on successful post; refs #195 — committed to caronc/apprise by caronc 4 years ago
I would also add that this picture seems fitting for this issue:
No worries; it’s my bad for not seeing this. I think @cstuder had found it originally and unable to even reproduce it then, i closed it off prematurely. I apologize for that sincerely. I’m glad it’s fixed now.
thanks as always!
Finally closing this issue off for good now; fix was merged into code base for next release.
Thanks @dgtlmoon, not sure why/when/how that changed, but I’ll get a fix in for you
Understood. We’ve raised the log level to ERROR so that silences the bug for now. Thanks for this library!
Thanks for the prompt fix, I tested it with master and it works.
And no, I don’t mind. Thank you for this great library!
The ticket auto-closes when I merged the code; sorry about not giving you a chance to provide feedback before hand.
I’m quite certain I nailed this issue though, but if you want to check out the master and give it a test, I’d really appreciate it.
On a side note: Great job with apprise-ga; I hope you don’t mind me sourcing it on my mini-showcase wiki page! 🙂