Project

General

Profile

Actions

Patch #1370

closed

Ignore unreadable subversion directories (read disabled using authz)

Added by Sergio Talens-Oliag almost 16 years ago. Updated almost 16 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
SCM
Target version:
Start date:
2008-06-04
Due date:
% Done:

0%

Estimated time:

Description

While testing Redmine with an existing Subversion repository I've found that the browser fails to generate the full list of directories when there is an unreadable directory (it is not readable because the user used by redmine does not have the right on the repository authz file).

The problem appears because redmine stops processing the svn list --xml output when it processes the unreadable directory, as it does not provide the author and date elements for the last commit (the code throws an exception when updating the lastrev's time attribute, as it tries to get the text value of the date element, but date is nil).

I've patched the code to ignore directory entries that do not have the date element, that way the unreadable directories are not shown on the repository browser and the list of entries is generated correctly.

My patch is attached to this issue.

I have not tried, but probably another way of fixing the problem would be to test if the date is nil when computing the lastrev's time attribute, that will show the directory on the browser and will fail only when trying to browse it.


Files

subversion_skip_unreadable_dirs.patch (911 Bytes) subversion_skip_unreadable_dirs.patch Patch against redmine-0.7 branch (r1489) Sergio Talens-Oliag, 2008-06-04 12:47
Actions #1

Updated by Jean-Philippe Lang almost 16 years ago

  • Status changed from New to Closed

Patch applied in r1500. Thanks.

Actions

Also available in: Atom PDF