Opened 19 years ago
Closed 19 years ago
#1327 closed defect (fixed)
Trac from trunk deadlocks after giving permission error
| Reported by: | Owned by: | Jonas Borgström | |
|---|---|---|---|
| Priority: | normal | Milestone: | 0.9 |
| Component: | general | Version: | devel |
| Severity: | normal | Keywords: | |
| Cc: | Branch: | ||
| Release Notes: | |||
| API Changes: | |||
| Internal Changes: | |||
Description
When using Trac from trunk with an environment that the web server cannot write to, Trac deadlocks instead of displaying the expected "The web server user requires read _and_ write permission to the database" message. Running Trac from the terminal via SCRIPT_NAME=/cgi-bin/trac.cgi PATH_INFO=/ TRAC_ENV=~/.trac/ sudo -u www python2.5 /Library/WebServer/CGI-Executables/_trac.cgi makes it apparent that it hangs after the error has been displayed, but obviously before output is flushed to the browser in CGI mode. Much fiddling later, the problem was tracked down to the ConnectionWrapper.del method in trac.db. Patch is attached.
Attachments (2)
Change History (3)
by , 19 years ago
| Attachment: | trac-hang-on-permission-error.patch added |
|---|
by , 19 years ago
| Attachment: | trac-hang-on-permission-error.2.patch added |
|---|
Didn't notice the ReferenceError's that were being raised. Catch these, as they're harmless.
comment:1 by , 19 years ago
| Milestone: | → 0.9 |
|---|---|
| Resolution: | → fixed |
| Status: | new → closed |
Patch applied in [1378]. Thanks!
Ensure that self.cnx is set to prevent an infinite loop in getattr occurring inside del. Also, don't swallow all errors.