Cjson Array Decoding not in the Latest Openresty Bundle

horatiolamouche · 2017-12-14T19:45:28+00:00

Looks like it's resolved now. Thx!On Wednesday, 13 December 2017 12:24:40 UTC-5, Nelson, Erik - 2 wrote: Not sure if you want to do this...

Cjson Array Decoding not in the Latest Openresty Bundle

horatiolamouche

Hi,

I recently ran into the issue of decoding and re-encoding json arrays with cjson.

Looking around, the openresty/cjson fork mentions:

This fork of mpx/lua-cjson is included in the OpenResty bundle and includes a few bugfixes and improvements, especially to facilitate the encoding of empty tables as JSON Arrays.

The openresty/cjson fork includes the method `decode_array_with_array_mt`:

$ ag '_array_mt"'
/tmp/lua-cjson/lua_cjson.c
1439:        { "decode_array_with_array_mt", json_cfg_decode_array_with_array_mt },
1491:    lua_setfield(l, -2, "empty_array_mt");

The method `decode_array_with_array_mt` is not present in the latest openresty bundle 1.13.6.1:

$ ag '_array_mt"'
/tmp/openresty-1.13.6.1/bundle/lua-cjson-2.1.0.5/lua_cjson.c
1445:    lua_setfield(l, -2, "empty_array_mt");

Looking at the openresty repo, it appears that the latest cjson bundled will be 2.1.0.6rc2. Will that contain the array decoding patch?

Thanks,

thibaultcha

Hi,

This patch is in the openresty/lua-cjson master branch indeed, but not yet shipped in an official OpenResty release. It should be included in the next one, as per the current state of the openresty/openresty repository (do not ask me when, I do not know).

Meanwhile if this cannot wait, you can retrieve the patch manually and apply it yourself to the lua-cjson source bundled with the OpenResty tarball (and recompile it, replace the old cjson.so with the new one).

Alternatively, you could clone the openresty/lua-cjson master branch locally, go to your local OpenResty sources from the latest release, and place it under:

/path/to/openresty-1.xx.x.x/bundle/lua-cjson-2.1.0.x

And then recompile OpenResty (which compiles lua-cjson for you).

Hope the patch solves your lua-cjson issues eventually.

Cheers,
Thibault

On 12/14/17 3:45 AM, horatio...@gmail.com wrote:

Hi,

I recently ran into the issue of decoding and re-encoding json arrays with cjson.

Looking around, the openresty/cjson fork mentions:

This fork of mpx/lua-cjson is included in the OpenResty bundle and includes a few bugfixes and improvements, especially to facilitate the encoding of empty tables as JSON Arrays.

The openresty/cjson fork includes the method `decode_array_with_array_mt`:

$ ag '_array_mt"' /tmp/lua-cjson/lua_cjson.c 1439: { "decode_array_with_array_mt", json_cfg_decode_array_with_array_mt }, 1491: lua_setfield(l, -2, "empty_array_mt");

The method `decode_array_with_array_mt` is not present in the latest openresty bundle 1.13.6.1:

$ ag '_array_mt"' /tmp/openresty-1.13.6.1/bundle/lua-cjson-2.1.0.5/lua_cjson.c 1445: lua_setfield(l, -2, "empty_array_mt");

Looking at the openresty repo, it appears that the latest cjson bundled will be 2.1.0.6rc2. Will that contain the array decoding patch?

Thanks,

H.
.