Skip to content

docs: document what AuthConfig.Auth is - #3910

Open
davidhsingyuchen wants to merge 1 commit into
docker:masterfrom
davidhsingyuchen:doc-auth
Open

docs: document what AuthConfig.Auth is#3910
davidhsingyuchen wants to merge 1 commit into
docker:masterfrom
davidhsingyuchen:doc-auth

Conversation

@davidhsingyuchen

@davidhsingyuchendavidhsingyuchen commented Dec 7, 2022

Copy link
Copy Markdown

PR enables folks to understand what Auth is without the need to trace code to:

// decodeAuth decodes a base64 encoded string and returns username and password
funcdecodeAuth(authStrstring) (string, string, error) {

Signed-off-by: Hsing-Yu (David) Chen davidhsingyuchen@gmail.com

Signed-off-by: Hsing-Yu (David) Chen <davidhsingyuchen@gmail.com>
@davidhsingyuchen
davidhsingyuchen marked this pull request as ready for review December 7, 2022 05:39
@thaJeztah

Copy link
Copy Markdown
Member

Thank you!

I want to double-check how this field is used, to be sure it's documented correctly as there's been some changes over time, and I'm not sure currently if this would always be username:password. These fields would be used to authenticate with different registries, and Authentication is not the same for all of them (which probably is one of the the reasons the field hasn't been documented so far).

I did a first search in history, but didn't completely finish that yet, but let me copy my notes here (in case I want to continue searching or someone else wants to);

@thaJeztahthaJeztah added this to the 29.5.1 milestone May 14, 2026
@vvolandvvoland modified the milestones: 29.5.1, 29.5.2May 18, 2026
@thaJeztahthaJeztah modified the milestones: 29.5.2, 29.5.3May 20, 2026
@vvolandvvoland modified the milestones: 29.5.3, 29.6.0Jun 3, 2026
@vvolandvvoland modified the milestones: 29.6.0, 29.6.1Jun 18, 2026
@vvolandvvoland modified the milestones: 29.6.1, 29.7.0Jun 30, 2026
@vvolandvvoland modified the milestones: 29.7.0, 29.8.0Jul 30, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@davidhsingyuchen@thaJeztah@vvoland@thompson-shaun