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

Add CLI credentials from CLI or JSON file #60

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

remingtonc
Copy link

@remingtonc remingtonc commented Aug 14, 2019

EDIT: Lost internet, draft state, need to squash and overwrite commits.

Adds CLI options to pull credentials from CLI args or JSON file. This is simply to aid with general automation scripts, systems, etc. which don't implement gNMI interfaces but may want to interact via shell.

-insecure_username & -insecure_password

Username and password may be passed directly on the command line. This has some notion of conflict with -insecure despite not being related, but generally wanted to discourage this method relatively explicitly and -insecure_* prefix feels most effective.

-credentials_file

Specified per client.Credentials:

{
    "Username": "x",
    "Password": "y"
}

Extracted functionality from #49 to separate docs and these args. I do think the docs are handy separately at least! Will maintain separately if not desired to merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant