arwin at collab
Nov 1, 2012, 1:57 AM
Post #2 of 2
Re: [BUG] PROPPATCH delete (svn propdel) errors not returned
[In reply to]
Can somebody please review this.
I would like to know if the current behavior would ever be changed.
On 07/09/2012 06:11 PM, Arwin wrote:
> Hi All,
> I've filed Bug 53525
> <https://issues.apache.org/bugzilla/show_bug.cgi?id=53525> in the ASF
> On failure to execute a PROPPATCH to remove a property, the error is not returned to the caller.
> A comment in the code says:
> <code snippet>
> 1044: /*
> 1045: ** Delete the property. Ignore errors -- the property is there, or
> 1046: ** we are deleting it for a second time.
> 1047: */
> 1048: /* ### but what about other errors? */
> 1049: (void) (*propdb->db_hooks->remove)(propdb->db, &name);
> </code snippet>
> In subversion, property deletions are subject to authorization, and hook script execution.
> In the current state of things, regardless of the outcome of the PROPPATCH, the client receives a 2xx class response. I suggest the attached patch to fix this.
> Arwin Arni