vbatts/ntfy
1
0
Fork 0
Code Pull requests Activity

Merge pull request #469 from ollien/fetch-get-body

Browse source 
Fix bug where GET or HEAD action requests could not be made from the web client
This commit is contained in:
Philipp C. Heckel 2022-11-07 05:05:51 -05:00 committed by GitHub
commit f3174f822f
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
web/src/components/Notifications.js
View file

@ -399,7 +399,9 @@ const performHttpAction = async (notification, action) => {
const response = await fetch(action.url, {
method: action.method ?? "POST",
headers: action.headers ?? {},
body: action.body ?? ""
// This must not null-coalesce to a non nullish value. Otherwise, the fetch API
// will reject it for "having a body"
body: action.body
});
console.log(`[Notifications] HTTP user action response`, response);
const success = response.status >= 200 && response.status <= 299;