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

Make maximum depth configurable via CLI option #3063

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Commits on Aug 24, 2024

  1. Add a notion of maximum depth to the parser state and jq_state.

    It's not sufficient to just add the state to the parser, since we
    might call `jv_load_file` from a variety of other functions.
    
    A few helper functions defer to `DEFAULT_MAX_PARSING_DEPTH`, as they
    are given neither a `jq_state` nor `jv_parser`.
    
    Added tests in `tests/shtest` to confirm that the maximum depth limit
    is appropriately enforced.
    mgree authored and wader committed Aug 24, 2024
    Configuration menu
    Copy the full SHA
    c50afa1 View commit details
    Browse the repository at this point in the history
  2. fix typo in usage

    mgree authored and wader committed Aug 24, 2024
    Configuration menu
    Copy the full SHA
    21ac76e View commit details
    Browse the repository at this point in the history
  3. missing newline at end of file

    mgree authored and wader committed Aug 24, 2024
    Configuration menu
    Copy the full SHA
    1faec65 View commit details
    Browse the repository at this point in the history
  4. add --depth to manual.yml

    mgree authored and wader committed Aug 24, 2024
    Configuration menu
    Copy the full SHA
    897aff0 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    1e5e1d7 View commit details
    Browse the repository at this point in the history
  6. add mention of --depth in error msg

    mgree authored and wader committed Aug 24, 2024
    Configuration menu
    Copy the full SHA
    6c6e47f View commit details
    Browse the repository at this point in the history