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

Re: svn commit: r38567 - in branches/1.6.x: . subversion/mod_dav_svn

From: C. Michael Pilato <cmpilato_at_collab.net>
Date: Wed, 05 Aug 2009 10:25:57 -0400

Stephen Butler wrote:
> Author: sbutler
> Date: Wed Aug 5 06:47:59 2009
> New Revision: 38567
>
> Log:
> Merge r36720 from trunk.

[...]

> Modified: branches/1.6.x/subversion/mod_dav_svn/liveprops.c
> URL: http://svn.collab.net/viewvc/svn/branches/1.6.x/subversion/mod_dav_svn/liveprops.c?pathrev=38567&r1=38566&r2=38567
> ==============================================================================
> --- branches/1.6.x/subversion/mod_dav_svn/liveprops.c Wed Aug 5 06:42:19 2009 (r38566)
> +++ branches/1.6.x/subversion/mod_dav_svn/liveprops.c Wed Aug 5 06:47:59 2009 (r38567)
> @@ -466,7 +466,7 @@ insert_prop(const dav_resource *resource
> && resource->info->r->content_type)
> mime_type = resource->info->r->content_type;
> else
> - mime_type = ap_default_type(resource->info->r);
> + mime_type = "application/octet-stream";
>
> if ((serr = svn_mime_type_validate(mime_type, p)))
> {

Sorry if/that I missed the discussion, but how was
"application/octet-stream" chosen as the appropriate substitution here? In
Apache 2.2, DEFAULT_CONTENT_TYPE is set (by default) to "text/plain", and
the default value of DefaultType configuration directive is "text/plain".
Did this change in 2.4?

-- 
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand
------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2380475

Received on 2009-08-05 16:26:25 CEST

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