Re: [PATCH 10/14] http: use hostname in credential description
- From
Jeff King <peff@peff.net>
- Date
- Aug 25, 2011, 20:23 UTC
- Message-ID
- <20110825202326.GC6165@sigill.intra.peff.net>
- In-Reply-To
- <87d3g1k3u6.fsf@lifelogs.com>
On Fri, Aug 19, 2011 at 07:01:21AM -0500, Ted Zlatanov wrote:
Show 6 quoted lines
> I see some info in "What's Cooking" about this patch but it's unclear to > me whether the hostname issue (where it's hard to have multiple > identities on a single server, which I think is all right) is blocking > the inclusion of the patch into the next release, or if it will be > included eventually if no one complains about that issue, or something > else...
Junio and I discussed it a bit in another thread. I think the ability to use "user@hostname" to disambiguate means the problem is dealt with at a high level. And the "cache" helper handles that just fine. But the "store" helper will conflate two entries for the same host. I'll see if I can work on a patch for that.
It looks like Junio is planning to hold the series off until 1.7.8. Have you been working on a Secrets API helper? If so, I'd love to get feedback on how well the interface is serving your needs.
-Peff