Skip to content

Conversation

@jc21
Copy link
Contributor

@jc21 jc21 commented Jan 22, 2026

This should be a Bearer token, not basic.

This should be a Bearer token, not basic.
@ffflorian
Copy link
Owner

Hi @jc21, thanks for the PR.

Where did you read that it should be Bearer?

@jc21
Copy link
Contributor Author

jc21 commented Jan 22, 2026

I actually tried it. I was getting unauthorised errors using token string and couldn't work out why until I saw your code. Basic auth is for user and pass. Bearer is the standard for tokens.

When I changed your source directly it worked.

If you want some examples, look for Bearer here: https://docs.ntfy.sh/examples/

@ffflorian ffflorian changed the title Fix ntfy Authorization using a Token fix(ntfy): Authorization using a Token Jan 22, 2026
@ffflorian
Copy link
Owner

ffflorian commented Jan 22, 2026

Basic auth is for user and pass. Bearer is the standard for tokens.

I know, but the docs explicitly state Basic authentication.

Did you use a private instance or the official instance?

@ffflorian
Copy link
Owner

ffflorian commented Jan 22, 2026

Ooh, I see. The publish docs are correct and provide more information.

Thank you, I will merge this now and see if I can improve this part of the API client.

@ffflorian ffflorian merged commit b75c0d9 into ffflorian:main Jan 22, 2026
4 checks passed
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.

2 participants