Project

General

Profile

Actions

Bug #32

closed

nocheinmal wrong json

Added by Anonymous 7 months ago. Updated 7 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Target version:
Start date:
03/03/2024
Due date:
% Done:

100%

Estimated time:
No private data included:
Yes

Description

Hi,

Ich mache mal einen neuen Issue auf, da ich mit der Antwort nicht ganz zufrieden bin
Ich bin eigentlich ganz deiner Meinung, dass integer bei json vollkommen egal sein sollte, aber es kommt sicherlich darauf an wie es der Server interpretiert. Mit curl kann ich es nachstellen (bei meinem eigenen Server und auch bei ntfy.sh direkt):

root@bl041:~# curl https://ntfy.sh/ -d '{"title":"Test-TITEL","message":"Test-Meldung","priority":3,"topic":"FreundlichenGruesseAnAlleFHEMNutzer"}'
{"id":"L0zowirDldR3","time":1709455312,"expires":1709498512,"event":"message","topic":"FreundlichenGruesseAnAlleFHEMNutzer","title":"Test-TITEL","message":"Test-Meldung","priority":3}

root@bl041:~# curl https://ntfy.sh/ -d '{"title":"Test-TITEL","message":"Test-Meldung","priority":"3","topic":"FreundlichenGruesseAnAlleFHEMNutzer"}'
{"code":40024,"http":400,"error":"invalid request: request body must be valid JSON"}

genau das verhalten sehe ich auch bei deinem Modul. Bei einer String-Uebergabe der Prioritaet erhalte ich keine Reaktion, auch keine Fehlermeldung, das Notify kommt nur nicht an. Wenn ich in dem Code priority als int uebergebe, wird das Notify sofort uebertragen.
Ich kann mir icht vorstellen, warum das evtl. nur bei mir so sein sollte, ist aber wie du oben siehst reproduzierbar. Ich nutze ein Debian Bookworm mit Perl v5.36.0.
Vielleicht kannst Du die Konvertierung zum integer evtl. doch einbauen, es wird meine Meinung nach keinem stoeren nur ich koennte mir die Arbeit sparen bei jedem Update den Code anzupassen. Und wer weis, vielleicht gibt es noch mehrere mit dem Problem

jedenfalls vielen Dank fuer das Modul, gefaellt mir sehr.
VG
Hartmut


Related issues 1 (0 open1 closed)

Is duplicate of FHEM Ntfy - Bug #19: wrong json format -> priority must be a number not a stringClosedDominik Meyer02/26/2024

Actions
Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0Like0
Go to top