RE: [PATCH 3/3] atomic-revprop: Signal the error as a HTTP status code

[Jon Foster <Jo...@cabot.co.uk>]

Hi,

> just two comments without having looked at the remainder of the
discussion.

Please read my first mail in this [PATCH 3/3] thread, which explains
why these choices were made.

Kind regards,

Jon


-----Original Message-----
From: Julian Reschke [mailto:julian.reschke@gmx.de] 
Sent: 21 September 2010 12:46
To: Jon Foster
Cc: Daniel Shahaf; Subversion Development
Subject: Re: [PATCH 3/3] atomic-revprop: Signal the error as a HTTP
status code

Hi,

just two comments without having looked at the remainder of the
discussion.

1) If you need to augment a standard HTTP response code with additional 
information, the right thing to use is DAV:error (see 
<http://greenbytes.de/tech/webdav/rfc4918.html#rfc.section.16>).

2) Do not use 412 Precondition Failed unless the request was a 
conditional request.

Best regards, Julian

______________________________________________________________________
This email has been scanned by the MessageLabs Email Security System.
For more information please visit http://www.messagelabs.com/email 
______________________________________________________________________


**********************************************************************
This email and its attachments may be confidential and are intended solely for the use of the individual to whom it is addressed. Any views or opinions expressed are solely those of the author and do not necessarily represent those of Cabot Communications Ltd.

If you are not the intended recipient of this email and its attachments, you must take no action based upon them, nor must you copy or show them to anyone.

Cabot Communications Limited
Verona House, Filwood Road, Bristol BS16 3RY, UK
+44 (0) 1179584232

Co. Registered in England number 02817269

Please contact the sender if you believe you have received this email in error.

**********************************************************************


______________________________________________________________________
This email has been scanned by the MessageLabs Email Security System.
For more information please visit http://www.messagelabs.com/email 
______________________________________________________________________

Re: [PATCH 3/3] atomic-revprop: Signal the error as a HTTP status code

[Julian Reschke <ju...@gmx.de>]

On 21.09.2010 13:52, Jon Foster wrote:
> Hi,
>
>> just two comments without having looked at the remainder of the
> discussion.
>
> Please read my first mail in this [PATCH 3/3] thread, which explains
> why these choices were made.
> ...

That's the message I read, and I didn't see any consideration of using 
DAV:error...

Best regards, Julian