mirror of
https://github.com/binwiederhier/ntfy.git
synced 2026-05-09 08:26:00 +02:00
[GH-ISSUE #630] Query parameters accept only lowercase parameter names #471
Labels
No labels
ai-generated
android-app
android-app
android-app
🪲 bug
build
build
dependencies
docs
enhancement
enhancement
🔥 HOT
in-progress 🏃
ios
prio:low
prio:low
pull-request
question
🔒 security
server
server
unified-push
web-app
website
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/ntfy#471
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @bbaa-bbaa on GitHub (Feb 22, 2023).
Original GitHub issue: https://github.com/binwiederhier/ntfy/issues/630
The parameter names are described as case-insensitive in the documentation, but only lowercase is accepted when passed by the query parameters
@wunter8 commented on GitHub (Mar 7, 2023):
Is there a specific query parameter you're having problems with? All of the following work fine for me.curl "https://ntfy.sh/mytopic/publish?message=testing"curl "https://ntfy.sh/mytopic/publish?Message=testing"curl "https://ntfy.sh/mytopic/publish?mEsSAgE=testing"Never mind. I didn't notice that the message was
triggeredinstead oftestingwhen using the capital query parameters.@binwiederhier commented on GitHub (Mar 7, 2023):
As per https://github.com/binwiederhier/ntfy/pull/661#issuecomment-1458700198, I think we're not going to do this. Instead, we'll just update the docs to reflect that query params are lower-case only.
@binwiederhier commented on GitHub (Mar 12, 2023):
Changed in the docs:
@wunter8 commented on GitHub (Mar 12, 2023):
lgtm