Thread (1 message) 1 message, 1 author, 2024-06-05

Re: [PATCH v2] credential: clear expired c->credential, unify secret clearing

From: Junio C Hamano <hidden>
Date: 2024-06-05 05:12:54

Rahul Rameshbabu [off-list ref] writes:
quoted
quoted
Aaron Plattner [off-list ref] writes:
...
Rahul (CC'd) and I had a series of patches to add something similar to the
current authtype system but hadn't gotten around to sending them to the list
before this more flexible mechanism was merged. It's nice that this worked out
of the box with minimal adjustment.

The credential helper he wrote is specific to the Microsoft "Entra ID" identity
provider system, but hopefully it'll be generally useful once this stuff is in a
git release. It really cleans up the authentication process over https for sites
that support it.
Aaron made a commit to make it work with the authtype/credential
credential-helper infrastructure that landed in git-next.

  https://github.com/Binary-Eater/git-credential-msal/commit/f71ca9c72ca1a2cf73373de76909f6007ac689cb

The support for authtype excites me since a number of large Git
providers like GitHub/GitLab/etc. have utilized Authorization Basic
incorrectly for supporting different authtypes with git previously.
Hoping they will move away from this practice in the future with this
enhancement.
Thanks for a huge praise that I do not personally deserve ;-)  Kudos
go to brian and folks who helped reviewing his topic.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help