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.

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

switch the json backend to json-iterator #19

Closed
wants to merge 1 commit into from

Commits on Aug 4, 2019

  1. switch the json backend to json-iterator

    add a couple of json interator configurations:
    
    1) jsonIterator should be compatible with the old encoding/json
    there is a breaking change here because the "JSONOpt" is now
    a no-op, although the options in encoding/json are not very useful.
    
    2) caseSensitiveStrictJSONIterator is compatible with
    econding/json WRT to DisallowUnknownFields, but it also has
    CaseSensitive. This is a breaking change for strict unmarshal users
    that previously allowed case-insensitive fields.
    
    Something else to note is that json-iter does not seem to tolerate
    field keys called "true", which the one from stdlib does.
    Unit tests had to be adapted because of that.
    
    Other changes:
    - add a new public method UnmarshalWithConfig that allows passing
    a json-iter configuration
    - update/add unit tests
    - update go.mod/sum
    - remove yaml_go110*.go
    These files had the purpose to handle DisallowUnknownFields for
    json.Decoder which is no longer needed.
    neolit123 committed Aug 4, 2019
    Configuration menu
    Copy the full SHA
    d8dcdb8 View commit details
    Browse the repository at this point in the history