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

proto1: update to operate over stdin #36

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

proto1: update to operate over stdin #36

wants to merge 2 commits into from

Commits on Feb 15, 2024

  1. proto1: parse arguments as JSON from a file 

    It's possible for proto1's command line interface to exceed the OS's
maximum argument length (128 KiB) if the JSON values passed into the
command are particularly large.

So instead of getting arguments from the command line, we parse them as
an equivalent JSON structure from a provided file. By default, this is
from stdin, but this can be overridden with the --input-file flag.

Signed-off-by: Sam Berning <bernings@amazon.com>
    @sam-berning
    sam-berning committed Feb 15, 2024
    Configuration menu
    Copy the full SHA
    d31196c View commit details
    Browse the repository at this point in the history

  2. tests: update cli interface to operate on a file 

    Signed-off-by: Sam Berning <bernings@amazon.com>
    @sam-berning
    sam-berning committed Feb 15, 2024
    Configuration menu
    Copy the full SHA
    e48be3d View commit details
    Browse the repository at this point in the history