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

Disallow nested objects and arrays as keys in objects #772

Merged
merged 4 commits into from Oct 1, 2023
Merged

Disallow nested objects and arrays as keys in objects #772

merged 4 commits into from Oct 1, 2023

Commits on Sep 20, 2023

  1. Generalize the logic to disallow nested objects and arrays as keys in… 

    … objects.

Fixes stleary#771.
    @eamonnmcmanus
    eamonnmcmanus committed Sep 20, 2023
    Configuration menu
    Copy the full SHA
    661114c View commit details
    Browse the repository at this point in the history

Commits on Sep 27, 2023

  1. Simplify the check for object keys that are themselves objects. 

    For object keys, we can just skip the part of `nextValue()` that parses values
that are objects or arrays. Then the existing logic for unquoted values will
already stop at `{` or `[`, and that will produce a `Missing value` exception.
    @eamonnmcmanus
    eamonnmcmanus committed Sep 27, 2023
    Configuration menu
    Copy the full SHA
    16967f3 View commit details
    Browse the repository at this point in the history

Commits on Sep 28, 2023

  1. Add more test cases for unquoted text in objects and arrays.

    @eamonnmcmanus
    eamonnmcmanus committed Sep 28, 2023
    Configuration menu
    Copy the full SHA
    dbb1131 View commit details
    Browse the repository at this point in the history

  2. Merge branch 'stleary:master' into complexkey

    @eamonnmcmanus
    eamonnmcmanus committed Sep 28, 2023
    Configuration menu
    Copy the full SHA
    eaa5611 View commit details
    Browse the repository at this point in the history