Skip to content

Update artifactory authentication flow#10

Open
redbugz wants to merge 2 commits intomasterfrom
update-artifactory-info
Open

Update artifactory authentication flow#10
redbugz wants to merge 2 commits intomasterfrom
update-artifactory-info

Conversation

@redbugz
Copy link
Copy Markdown

@redbugz redbugz commented Apr 9, 2026

Summary

Improve GitHub credential validation during setup by using git credential fill instead of direct .netrc parsing. This provides better fallback support for credential sources including macOS keychain.

Changes

  • Update GitHub authentication to use git credential helper
  • Supports multiple credential sources (keychain, .netrc, etc.)
  • Cross-platform compatible (Windows, macOS, Linux)
  • Better error messaging for setup validation

🤖 Generated with Claude Code

Improvements to setup and verification:

GitHub Credentials:
- Use git credential helper instead of direct .netrc parsing
- Automatically falls back through multiple credential sources (keychain, .netrc, etc)
- Works cross-platform (Windows, macOS, Linux)
- Rename checkNetrcConfig to checkGitHubAccess to reflect broader scope

Artifactory Setup:
- Simplify setup to use npm config set and interactive npm login
- Browser-based SAML SSO authentication flow
- Verify registry is correctly configured before proceeding
- Use npm config get instead of grepping .npmrc for consistency
- Both setup and check scripts use npm config uniformly

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
@redbugz redbugz force-pushed the update-artifactory-info branch from 98f9b1d to e03e13e Compare April 10, 2026 20:30
@redbugz redbugz requested review from a team and joeycozza April 10, 2026 22:02
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