varnish / libvmod-rtstatus

Varnish realtime status page.
Other
37 stars 16 forks source link

Trailing comma in be_info array #26

Closed knuthaug closed 8 years ago

knuthaug commented 8 years ago

Hi

Trying to parse the json output from rtstatus.json all json parsers I have tried complain about the trailing comma in the array of backends (be_info). AFAIK the json spec also disallows trailing commas in arrays. Any chance of a quick fix for this?

daghf commented 8 years ago

@aondio please have a look at this when you get around to it

aondio commented 8 years ago

Hi @knuthaug, which branch are you using?

knuthaug commented 8 years ago

I have no idea, installed by the guys in Redpill-Linpro. Maybe @JoarJ or @obliadp knows?

aondio commented 8 years ago

Hi, Can you also please tell me how you define backends and directors?

I've tried to reproduce this with VC4.1 and using the branch 4.1 of this vmod and I didn't encounter any error.

obliadp commented 8 years ago

try VCP4.1 :-)

aondio commented 8 years ago

I've discussed this on IRC with Orjan(not sure which is his github name) and turned out that they where running a too old version of this vmod. Branch 4.1, starting from commit 3fa94e7 is no longer affected. Closing.