Modify ↓
Opened 12 years ago
Closed 12 years ago
#10793 closed defect (fixed)
TracError if a repo is badly configured
Reported by: | ejucovy | Owned by: | ejucovy |
---|---|---|---|
Priority: | high | Component: | MultiRepoSearchPlugin |
Severity: | major | Keywords: | |
Cc: | Trac Release: |
Description
To reproduce:
- Add a repo to Trac
- Use the multireposearchplugin's sqlindexer to index the repo's files
- Remove the repo from the filesystem, but leave it configured in Trac
- Search for a term that exists in the repo's file contents
The system will throw a different error depending on the version control backend used. For Git, it will look like /path/to/repository.git/ does not appear to be a Git repository.
Instead of throwing a system error, this exception should be caught and logged, and the offending search result should be ignored.
Attachments (0)
Note: See
TracTickets for help on using
tickets.
Fixed in https://github.com/boldprogressives/trac-MultiRepoSearchPlugin/commit/e8999159a3df932c45a165fc9d3c8685adb94f46