Project

General

Profile

Actions

Bug #32

closed

nocheinmal wrong json

Added by Anonymous 9 months ago. Updated 9 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 #1

Updated by Dominik Meyer 9 months ago

  • Is duplicate of Bug #19: wrong json format -> priority must be a number not a string added
Actions #2

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

Actions #3

Updated by Dominik Meyer 9 months ago

  • Target version changed from Future Work to 1.0.0
Actions #4

Updated by Dominik Meyer 9 months ago

  • Status changed from New to Accepted
  • % Done changed from 0 to 10
Actions #5

Updated by Dominik Meyer 9 months ago

  • Status changed from Accepted to Closed
  • % Done changed from 10 to 100
Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0Like0