Skip to content

Update RegistryAuthentication.md#634

Open
baronfel wants to merge 2 commits into
mainfrom
baronfel-patch-1
Open

Update RegistryAuthentication.md#634
baronfel wants to merge 2 commits into
mainfrom
baronfel-patch-1

Conversation

@baronfel

@baronfel baronfel commented Jun 5, 2025

Copy link
Copy Markdown
Member

Added some missing env vars to the auth section

@baronfel baronfel requested a review from a team June 5, 2025 20:15
* DOTNET_CONTAINER_PULL_REGISTRY_PWORD
* This should be the password, token, etc for the _source_ registry.

_Supported but not recommended_

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should we document the precedence behavior when both sets of variables are set?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

It's:

  • preferred variants always supersede the older variants if present
  • both username and password must come from the same 'variant'

So

DOTNET_CONTAINER_PUSH_REGISTRY_UNAME=...
DOTNET_CONTAINER_PUSH_REGISTRY_PWORD=...
DOTNET_CONTAINER_REGISTRY_UNAME=....
DOTNET_CONTAINER_REGISTRY_PWORD=....

would use DOTNET_CONTAINER_PUSH_REGISTRY_UNAME/DOTNET_CONTAINER_PUSH_REGISTRY_PWORD for the Push side of any operations, and
DOTNET_CONTAINER_REGISTRY_UNAME/DOTNET_CONTAINER_REGISTRY_PWORD for the Pull side of any operations.

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.

2 participants