Python Backend Allows Leading 0s in Numbers about ijson HOT 5 CLOSED

A new 3.1.3 release containing this fix and others is now up in PyPI.

from ijson.

Thanks @lwoerdeman for reporting this. This was indeed a wrong behavior on the python backend, which I have now fixed and will soon merge back to the master branch (and eventually produce a new release on PyPI, together with other fixes).

While fixing this (and adding tests to ensure it didn't occur in the future) I found two more small problems:

• JSON numbers containing a decimal point need to be surrounded by at least one digit at each side (e.g., .1 and 1. are invalid). The python backend also incorrectly allowed these, so I fixed that as well.
• The yajl backend also allows leading zeros. Note that this backend uses yajl version 1.x and has been considered deprecated for a long time. To address this I simply documented the difference in behvaior on the README file.

from ijson.

Fix is now on the master branch.

from ijson.

Hey @rtobar, thank you for resolving this issue so quickly. Do you have an expected release date for the fix to be in PyPI?

from ijson.

@lwoerdeman hopefully within one or two days. There is another minor fix I want to include before creating a new patch release, but I need to first make sure it works as expected.

from ijson.