GNU bug report logs - #72941
jsonrpc: Check if parameters are in line with the spec

Previous Next

Package: emacs;

Reported by: Felician Nemeth <nemethf <at> tmit.bme.hu>

Date: Sun, 1 Sep 2024 21:48:03 UTC

Severity: normal

Done: Eli Zaretskii <eliz <at> gnu.org>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Felician Nemeth <nemethf <at> tmit.bme.hu>
To: Daniel Pettersson <daniel <at> dpettersson.net>
Cc: Eli Zaretskii <eliz <at> gnu.org>, João Távora <joaotavora <at> gmail.com>, 72941 <at> debbugs.gnu.org
Subject: bug#72941: jsonrpc: Check if parameters are in line with the spec
Date: Fri, 13 Sep 2024 19:13:16 +0200
>>> Would it be possible to extend jsonrpc.el to check the params argument
>>> of jsonrpc-request, jsonrpc-notify, and jsonrpc-async-request whether it
>>> is a structured value?  And if it is not, then guide the programmer to
>>> the above URL with a warning.
>
> The use case makes sense to me, but I would go with updating the docs
> rather then the API.  The current wording could use some love as it's
> refereed to as JSON object or plist (at different functions).  When it
> should be plist or vector, where we could throw in a link to the
> specification.

Makes sense.

> I am not for signaling an error or similar as it's a breaking change in
> my book, which does not seam called for in this case.

Maybe when the server responds with an error to a jsonrpc-request, then
jsonrpc.el could create an additional warning if the params of the
request was not structured.  Or maybe it is too much work for a very
small gain.

Thank you.




This bug report was last modified 221 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.