Re: [PATCH] daemon: return "access denied" if a service is not allowed

On Thu, Oct 13, 2011 at 4:59 PM, Jonathan Nieder [off-list ref] wrote:

Nguyen Thai Ngoc Duy wrote:

quoted

How about allow users to select which messages they want to print? We
can even go further, allowing users to specify the messages themselves..

[...]

quoted

+     { "service not enabled", "message.serviceNotEnabled" },
+     { "no such repository", "message.noSuchRepository" },
+     { "repository not exported", "message.repositoryNotExported" },

I administer a private server that is only accessible as "localhost".
:)  This much customization would leave me confused about what the
right choices are and what the choices mean (even if I were to make
the server public and start having security worries).

--informative-errors is your friend. All errors are enabled.

What is the intended use --- translation?  The idealist in me thinks
that should be taken care of on the client side, if at all.  (This
way, we would not be preventing especially friendly clients from
offering pertinent detailed advice for each error condition.
Alternatively, maybe some day the protocol will want to provide a way
for clients to indicate a preferred language and message verbosity.)

Translation could be fun to do, but it's more about how much admins
want to reveal. For example, I may only want to show "service not
enabled" and "no such repository", not the last one, which simply
becomes "access denied".

Again I'm not real admin and this may be just bogus.
-- 
Duy