Re: [PATCH/RFC] Add separate error code for RA Forbidden

Ping. This patch hasn't had any responses yet.

On 05/03/2009, at 1:41 PM, Jelmer Vernooij wrote:

> At the moment a generic error (175002,
> SVN_ERR_RA_DAV_REQUEST_FAILED) is
> returned when ra_neon hits a 403 error, and this makes it hard to
> distinguish it from other RA errors that also return 175002.
>
> [[[
>
> * subversion/include/svn_error_codes.h: Add error code
> SVN_ERR_RA_FORBIDDEN.
> * subversion/libsvn_ra_neon/util.c(generate_error): Return
> SVN_ERR_RA_FORBIDDEN for 403 HTTP codes.
>
> ]]]
>
> Is SVN_ERR_RA_FORBIDDEN the right name here and RA the right category,
> or does it belong in the DAV specific error category?
>
> Cheers,
>
> Jelmer
>
> ------------------------------------------------------
> http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1270092
> <svn_ra_neon_forbidden.diff>

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1313340
Received on 2009-03-12 16:25:54 CET