Feature #16502: Add option :allow_trailing_comma to JSON#parse

• Status: Closed
• Assignee: -
• Target version: -

[j.spanjers (Jan-Joost Spanjers)]

Hello everyone,

I'd like to be able to parse JSON sources that contain trailing commas in arrays and/or objects.

For example,

> JSON.parse('[1,2,3,]')
JSON::ParserError: 416: unexpected token at ']'

I have created a pull request to add the option :allow_trailing_comma to JSON#parse:

> JSON.parse('[1,2,3,]', allow_trailing_comma: true)
[1, 2, 3]

I would kindly like to ask you to consider this patch for merging in Ruby core.

Regards,
Jan-Joost Spanjers

[1] https://github.com/ruby/ruby/pull/2831

[mame (Yusuke Endoh)]

• Status changed from Open to Feedback

The json library has another upstream repository: https://github.com/flori/json

Could you please send a pull request to them?

[j.spanjers (Jan-Joost Spanjers)]

mame (Yusuke Endoh) wrote:

The json library has another upstream repository: https://github.com/flori/json

Could you please send a pull request to them?

Thank you for your feedback. I have created a new pull request (https://github.com/flori/json/pull/401) upstream.

[mame (Yusuke Endoh)]

• Status changed from Feedback to Closed

Thanks! If the pull request is merged, it will be backported to ruby/ruby. So I close this ticket.