Skip to content

Fix Docker Hub PAT auth flow#79

Open
RomanOstr wants to merge 1 commit into
docker:mainfrom
Ceratops-Code:codex/upstream-pat-auth-fix
Open

Fix Docker Hub PAT auth flow#79
RomanOstr wants to merge 1 commit into
docker:mainfrom
Ceratops-Code:codex/upstream-pat-auth-fix

Conversation

@RomanOstr
Copy link
Copy Markdown

Summary

  • exchange Docker Hub PATs through the documented /v2/auth/token endpoint
  • trim PAT input before exchange so secrets stored with a trailing newline still authenticate
  • make the tools-list checker tolerate CRLF in tools.txt
  • add Git attributes for LF-normalized source/check files to match the existing Prettier config

Validation

  • npm run format:check
  • npm run build
  • npm run list-tools:check
  • MCP smoke test: getPersonalNamespace returns the authenticated namespace with a PAT

@RomanOstr RomanOstr requested a review from a team as a code owner April 17, 2026 21:04
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