From: Junio C Hamano Date: Fri, 14 Oct 2011 20:48:51 GMT Subject: Re: [PATCH] daemon: return "access denied" if a service is not allowed Message-ID: <7vpqhzxpsc.fsf@alter.siamese.dyndns.org> In-Reply-To: <20111014203438.GA15643@sigill.intra.peff.net> Jeff King writes: >> It would have been a better split to have the 1/2 patch to support both >> informative and uninformative errors, with the default to say "access >> denied", and 2/2 to flip the default to be more open. > > Isn't that what I did? It was what I meant to do, anyway... > > Or did you mean the options would have been better worded as: > > --errors={terse,informative} > > or something similar? Nothing that elaborate. Supporting --no-* variant even when the default is already no will allow people to prepare their daemon invocation command line beforehand to ensure that they won't be affected to a more lenient default that may or may not come in the future. That's all.