[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

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

From: Daniel Shahaf <d.s_at_daniel.shahaf.name>
Date: Tue, 21 Sep 2010 14:12:12 +0200

Julian Reschke wrote on Tue, Sep 21, 2010 at 13:46:13 +0200:
> 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>).
>

What does that gain us? As Jon said elsethread, generic DAV clients
aren't going to be making requests with V:old-value set.

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

FTR, cmpilato suggested to use 409 Conflict at some point.

> Best regards, Julian
Received on 2010-09-21 14:13:23 CEST

This is an archived mail posted to the Subversion Dev mailing list.