Skip to content

Apply work identity to HTTPS work-org remotes - #18

Open
hadees wants to merge 2 commits into
masterfrom
fix/gitconfig-https-work-identity
Open

Apply work identity to HTTPS work-org remotes#18
hadees wants to merge 2 commits into
masterfrom
fix/gitconfig-https-work-identity

Conversation

@hadees

@hadees hadees commented Jul 25, 2026

Copy link
Copy Markdown
Owner

Why

The includeIf hooks in dot_gitconfig.local.tmpl only matched the github-icaevan: SSH alias, but work repos cloned over HTTPS (icanalytica standardizes on HTTPS remotes) fell through to the personal identity — commits landed with the wrong email and signing key.

What

Add two more includeIf "hasconfig:remote.*.url:..." rules matching https://github.com/icanalytica/** and https://github.com/icarichie/**, mirroring the org list the credential-routing section below already pins to icaevan. Requires git 2.36+ like the existing rules.

Already applied live on this machine and verified: an HTTPS icanalytica clone now resolves user.email to the work address, github.user to icaevan, and the icaevan signing key. chezmoi diff shows source and target in sync for this file.

🤖 Generated with Claude Code

The includeIf hooks only matched the github-icaevan SSH alias, but work
repos cloned over HTTPS (icanalytica standardizes on HTTPS remotes) fell
through to the personal identity — wrong email and signing key on
commits. Match https://github.com/icanalytica/** and icarichie/** too,
same as the credential-routing section below already does.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@hadees hadees self-assigned this Jul 25, 2026
Credentials stay in 1Password; ~/.config/namecheap/env holds only op://
secret references, resolved per-invocation by op run. Ephemeral machines
skip the file (no 1Password there).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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