alan5281 / cometd

Automatically exported from code.google.com/p/cometd
0 stars 0 forks source link

bayeux.json and bayeux_error_codes.json broken in trunk/ #15

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Checkout http://svn.cometd.com/trunk/
2. Run twistd -noy cometd.tac in cometd-twisted/

Everything breaks in cometd-twisted/bayeux.py, line 94 with
JSON decoding errors.

Indeed, after checking bayeux/bayeux.json and bayeux/bayeux_error_codes.json, I 
can confirm that 
they do not follow JSON syntax. There are a lot of missing commas.

One other issue is that simplejson (Python library used to decode JSON) somewhy 
doesn't recognize 
000 as a number, so I also had to convert those to plain 0. This can't 
generally hurt, except for 
readability.

The patch against r480 is attached.

Original issue reported on code.google.com by adomas.p...@gmail.com on 4 Jun 2008 at 10:58

Attachments:

GoogleCodeExporter commented 9 years ago
numbers are not allowed to start with a 0 unless it is 0 or beginning of a 
decimal number.  000 is invalid.
read json.org for more info

This problem was annoying. I wish someone would add the patch in.

Original comment by zectb...@gmail.com on 14 Dec 2010 at 3:53

GoogleCodeExporter commented 9 years ago
patch applied to cometd-2 r1576.  
Sorry for the long delay.   

Note that the contents of the bayeux.json file have not been verified.

Original comment by gregory....@gmail.com on 15 Dec 2010 at 11:13