Am 04/09/2017 10:34 AM, Olivier Mascia wrote:
Le 9 avr. 2017 à 03:08, Jens Alfke <j...@mooseyard.com> a écrit :
On Apr 7, 2017, at 5:26 PM, Rolf Ade <r...@pointsman.de> wrote:
./sqlite3
SQLite version 3.19.0 2017-04-07 20:20:08
[...]
sqlite> select json(' { "this" : 000.23 } ');
{"this":000.23}
If I read RFC 7159 (http://www.rfc-editor.org/rfc/rfc7159.txt) correct
this should return: "Error: malformed JSON".
In this case I would go with Postel’s Law, paraphrased as “Be
strict in what you write, but lenient in what you read.” I don’t see
a point in disallowing something as trivial as redundant leading
zeroes.
Mr. Hipp has already fixed this:
https://www.sqlite.org/src/info/204e72f0080e8f08
If you think, that Postel's law should applied here, then Olivier
already pointed out rightfully:
If you'd go with Postal's Law, you would make it so:
sqlite> select json(' { "this" : 000.23 } '); // be lenient in what you
read
{"this":0.23} // be strict in what you write
I think, you do your users no good on the long run, if you accept not
recommendation compliant input (without explict request to do that by
the user). After all, JSON isn't a very complex standard and the
specification does not let much room (if ever) to argue if a certain
input string is valid or not. And JSON isn't an internet protocol, but
a data interchange format.
That all said I'm far from being religious about this. Even a "won't
fix" or a "works as designed" would have been OK with me.
This even wasn't a case I god bitten by this in the wild. For another
project I'm currently writing my 'own' JSON parser. To do that, I took
a look at the sqlite JSON parser implementation (just because I knew
it's on my hard disk and Mr. Hipps code is high quality). I stumbled
over this just by studying the sqlite json1.c code and wanted to make
sure, this implemenation detail is known and decided deliberately.
_______________________________________________
sqlite-users mailing list
sqlite-users@mailinglists.sqlite.org
http://mailinglists.sqlite.org/cgi-bin/mailman/listinfo/sqlite-users
