Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Document the NOTARY_AUTH environment variable #1309

Merged
merged 1 commit into from
Mar 22, 2018

Conversation

cyli
Copy link
Contributor

@cyli cyli commented Mar 2, 2018

Documents the environment variable introduced in #1246. Fixes #1292.

Signed-off-by: cyli <cyli@twistedmatrix.com>
Copy link
Contributor

@riyazdf riyazdf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice!

@@ -185,6 +185,7 @@ passphrase.
|`NOTARY_TARGETS_PASSPHRASE` | The targets (an online) key passphrase |
|`NOTARY_SNAPSHOT_PASSPHRASE` | The snapshot (an online) key passphrase |
|`NOTARY_DELEGATION_PASSPHRASE` | The delegation (an online) key passphrase |
|`NOTARY_AUTH` | The notary server creds ("username:password"), base64-ed |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The creds seems not used for notary server .

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It’s to authenticate against notary server

@justincormack
Copy link
Contributor

LGTM

@endophage endophage merged commit 80641bc into notaryproject:master Mar 22, 2018
@cyli cyli deleted the update-docs branch March 22, 2018 17:45
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants