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

Added support for Access Tokens and improved message conditions #1

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

TheNaubit
Copy link

First of all, thanks for building this, this was a really useful starting tool for integrating Ntfy into my system.

I needed a few changes and I wanted to share them in case it is useful for others. I added the following:

  • Support for access tokens. In case you have a secured Ntfy server, you might need to use it. You can now set the env var called NTFY_ACCESS_CONTROL with an access token you created (without the Bearer string), as explained here: https://docs.ntfy.sh/publish/#access-tokens If you are not in need of access tokens, you can just leave this var unset.
  • I had an issue where I needed to send a message without a title. So I modified the conditions so it ignores the message if there is no title and no message or both are empty strings. I also made it so that if there is a title, we send the title section of the message in bold, using MarkdownV2.

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

Successfully merging this pull request may close these issues.

None yet

1 participant