Bug #32
closednocheinmal wrong json
100%
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
Updated by Dominik Meyer 9 months ago
- Is duplicate of Bug #19: wrong json format -> priority must be a number not a string added
Updated by Dominik Meyer 9 months ago
Hi,
ich schau noch mal. Arbeite normalerweise mit json in C++ und da legt der Empfänger fest was er erwartet und konvertiert das passend. Offensichtlich macht ntfy.sh das anders.
Bei mir kamen bisher aber alle Nachrichten an.
bis denn
Dominik
Updated by Dominik Meyer 9 months ago
- Target version changed from Future Work to 1.0.0
Updated by Dominik Meyer 9 months ago
- Status changed from New to Accepted
- % Done changed from 0 to 10
Updated by Dominik Meyer 9 months ago
- Status changed from Accepted to Closed
- % Done changed from 10 to 100