Per discussion with cmpilato on IRC, the current plan is to stow the error
chain under the <D:responsedescription> element --- which is what the 207
Multi-Status response currently contains, and which currently contains only
the error message (but not its numeric code).
Daniel
(going to sleep, so short)
Daniel Shahaf wrote on Sat, Aug 07, 2010 at 22:24:38 +0300:
> <short version>
> In order to properly use the svn_ra_change_rev_prop2() API (on the
> 'atomic-revprop') branch, I'd like to make it return a specific error code
> (over all RA layers) when the propchange fails due to the provided expected
> value not being the found value. (That's kinda the point of the new API.)
>
> To do so, I'd like to have mod_dav_svn transmit the error code it saw to
> ra_neon/ra_serf; currently it swallows the error code and I don't see yet
> how to unswallow it.
>
> I see from the code that this should be possible using <svn:error/> and
> <human-readable errcode="%d"/> tags, but I don't see yet how.
> </short version>
>
>
>
> Over all RA layers, I get the following error message:
>
> "revprop 'flower' has unexpected value in filesystem"
>
>
> Over ra_local and ra_svn, the error chain contains the original error code
> (SVN_ERR_BAD_PROPERTY_VALUE, 125005):
>
> [[[
> % ./subversion/tests/cmdline/atomic-ra-revprop-change file://$PWD/subversion/tests/cmdline/svn-test-work/repositories/prop_tests-34 0 flower "( 11 old_value_p 11 wrong value 5 value 6 violet )" neon 1
> subversion/libsvn_repos/fs-wrap.c:320: (apr_err=125005)
> subversion/libsvn_fs_fs/fs_fs.c:609: (apr_err=125005)
> subversion/libsvn_fs_fs/fs_fs.c:609: (apr_err=125005)
> subversion/libsvn_fs_fs/fs_fs.c:7263: (apr_err=125005)
> atomic-ra-revprop-change: revprop 'flower' has unexpected value in filesystem
> zsh: exit 1 ./subversion/tests/cmdline/atomic-ra-revprop-change 0 flower neon 1
> ]]]
> [[[
> % ./subversion/tests/cmdline/atomic-ra-revprop-change svn://localhost/svn-test-work/repositories/prop_tests-34 0 flower "( 11 old_value_p 11 wrong value 5 value 6 violet )" neon 1
> subversion/libsvn_ra_svn/client.c:855: (apr_err=210000)
> subversion/svnserve/serve.c:1013: (apr_err=210000)
> atomic-ra-revprop-change: Special code for wrapping server errors to report to client
> subversion/libsvn_repos/fs-wrap.c:320: (apr_err=125005)
> atomic-ra-revprop-change: revprop 'flower' has unexpected value in filesystem
> subversion/libsvn_fs_fs/fs_fs.c:609: (apr_err=125005)
> atomic-ra-revprop-change: revprop 'flower' has unexpected value in filesystem
> subversion/libsvn_fs_fs/fs_fs.c:609: (apr_err=125005)
> atomic-ra-revprop-change: revprop 'flower' has unexpected value in filesystem
> subversion/libsvn_fs_fs/fs_fs.c:7263: (apr_err=125005)
> atomic-ra-revprop-change: revprop 'flower' has unexpected value in filesystem
> zsh: exit 1 ./subversion/tests/cmdline/atomic-ra-revprop-change 0 flower neon 1
> ]]]
>
> That's good.
>
> However, over ra_neon and ra_serf, while the the error chain contains the
> original message text, the only error codes available are
> SVN_ERR_RA_DAV_REQUEST_FAILED and SVN_ERR_RA_DAV_PROPPATCH_FAILED:
>
> [[[
> % ./subversion/tests/cmdline/atomic-ra-revprop-change http://localhost:8081/trunk1/svn-test-work/repositories/prop_tests-34 0 flower "( 11 old_value_p 11 wrong value 5 value 6 violet )" neon 1
> subversion/tests/cmdline/atomic-ra-revprop-change.c:199: (apr_err=175002)
> subversion/libsvn_ra_neon/fetch.c:1209: (apr_err=175002)
> atomic-ra-revprop-change: 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:1231: (apr_err=175008)
> atomic-ra-revprop-change: At least one property change failed; repository is unchanged
> subversion/libsvn_ra_neon/util.c:1506: (apr_err=175002)
> subversion/libsvn_ra_neon/util.c:212: (apr_err=175002)
> atomic-ra-revprop-change: Error setting property 'flower':
> revprop 'flower' has unexpected value in filesystem
> zsh: exit 1 ./subversion/tests/cmdline/atomic-ra-revprop-change 0 flower neon 1
> ]]]
> [[[
> % ./subversion/tests/cmdline/atomic-ra-revprop-change http://localhost:8081/trunk1/svn-test-work/repositories/prop_tests-34 0 flower "( 11 old_value_p 11 wrong value 5 value 6 violet )" serf 1
> subversion/tests/cmdline/atomic-ra-revprop-change.c:199: (apr_err=175002)
> subversion/libsvn_ra_serf/commit.c:2352: (apr_err=175002)
> atomic-ra-revprop-change: DAV request failed; it's possible that the repository's pre-revprop-change hook either failed or is non-existent
> subversion/libsvn_ra_serf/commit.c:229: (apr_err=175008)
> atomic-ra-revprop-change: At least one property change failed; repository is unchanged
> subversion/libsvn_ra_serf/util.c:1041: (apr_err=175002)
> atomic-ra-revprop-change:
> revprop 'flower' has unexpected value in filesystem
> subversion/libsvn_ra_serf/commit.c:229: (apr_err=175002)
> atomic-ra-revprop-change: PROPPATCH of '/trunk1/svn-test-work/repositories/prop_tests-34/!svn/rev/0': 207 Multi-Status
> zsh: exit 1 ./subversion/tests/cmdline/atomic-ra-revprop-change 0 flower serf 1
> ]]]
>
>
> Is it possible to make mod_dav_svn allow another (specific) apr_err error
> code to be passed over the wire to the client? I'd like the error chain on
> the ra_neon/ra_serf side to contain SVN_ERR_BAD_PROPERTY_VALUE (125005).
>
> Looking at `egrep 'apr_err = |svn_error_create' libsvn_ra_neon/*`[1] tells me
> that the answer should involve ELEM_svn_error (svn:error) and
> ELEM_human_readable (<human-readable errcode="%d"/>).
>
> Hints, please?
>
> Thanks,
>
> Daniel
>
>
>
> [1] libsvn_ra_neon/util.c:start_err_element() contains the only relevant match.
Received on 2010-08-09 22:53:32 CEST