Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

don't pass non-JSON specific producer_args to to_json #64

Closed
wants to merge 2 commits into from
Closed

don't pass non-JSON specific producer_args to to_json #64

wants to merge 2 commits into from

Commits on Aug 2, 2015

  1. don't pass non-JSON specific producer_args to to_json 

    The producer_args passed by sqlt contains a number of keys which are
not specific to JSON.  These were passed unfiltered to to_json().
JSON (at least as of v2.90) will throw an error if it is passed an
unknown option (it uses the option key as a method name, which leads
to confusing error messages).

It's not straightforward to automatically determine the args supported
by the JSON module, so this simply whitelist the 'pretty', 'indent',
and 'canonical' options.
    @djerius
    djerius committed Aug 2, 2015
    Configuration menu
    Copy the full SHA
    b18e61c View commit details
    Browse the repository at this point in the history

Commits on Sep 4, 2015

  1. remove introduced tabs

    @djerius
    djerius committed Sep 4, 2015
    Configuration menu
    Copy the full SHA
    97fd40c View commit details
    Browse the repository at this point in the history