Re: mod_dav_svn not XML-escaping hook error messages

Julian Foad <julianfoad_at_btopenworld.com> writes:

>> <http://subversion.tigris.org/issues/show_bug.cgi?id=3553>
>
> My test (the test patch that I posted a couple of messages back in this
> thread), running trunk without any patch to fix the bug, is producing:
>
> EXPECTED STDERR:
> svn: post-revprop-change hook failed (exit code 1) with output:
> post-revprop-change hook failed: Text with <angle brackets> & ampersand
> ACTUAL STDERR:
> subversion/svn/propset-cmd.c:127: (apr_err=175002)
> subversion/libsvn_client/prop_commands.c:485: (apr_err=175002)
> subversion/libsvn_ra_neon/fetch.c:1173: (apr_err=175002)
> svn: DAV request failed; it's possible that the repository's
> pre-revprop-change hook either failed or is non-existent
> subversion/libsvn_ra_neon/props.c:1158: (apr_err=175008)
> svn: At least one property change failed; repository is unchanged
> subversion/libsvn_ra_neon/util.c:613: (apr_err=175002)
> svn: PROPPATCH of '/svn-test-work/repositories/prop_tests-30/!
> svn/bln/0': 207 Multi-Status (http://localhost:28886)
>
> And 1.6.x branch does the same.
>
> That's not the error Jon reported: he saw an "invalid XML" error. Why
> the difference?

I don't know, but I get the 207 error you see (although only with
neon, with serf the client reports success even though the operation
failed).

-- 
Philip