> Filed as issue #4888, https://subversion.apache.org/issue/4888

I have just been looking back over this issue. Clearly there is more to
it than a quick fix. Summary, based on reviewing the email thread:

- FAIL: authz_tests.py 31 remove_access_after_commit

- Patched in <https://svn.apache.org/r1898184>: in "text base sync"
phase, ignore auth error while fetching any text base; continue with
trying to fetch the rest.

- While that change fixes that particular test case, it also seems to
have regressed the failure mode of «svn cat iota@BASE» when iota is
locally-modified and has no read access. On trunk that command displayed
the base text; on this branch that command now errors out.

- We started thinking about what other failure modes could now occur
because of failures (including unauthorised, redirect, and others) at
the hydrate phase. This is somewhat open ended; we don't have a simple
answer to how this all should be taken care of consistently.

I suggest:

- revert the patch I applied, as it's papering over the problem in an
incomplete way and so possibly causes more confusion than it fixes.

- leave this issue open and come back to it later; it's an edge case not
part of common work flows.

Reply via email to