Merge lp:~michael.nelson/ubuntu-webcatalog/render_404_rather_than_raise_exception into lp:ubuntu-webcatalog
Proposed by
Michael Nelson
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Michael Nelson | ||||
Approved revision: | 170 | ||||
Merged at revision: | 167 | ||||
Proposed branch: | lp:~michael.nelson/ubuntu-webcatalog/render_404_rather_than_raise_exception | ||||
Merge into: | lp:ubuntu-webcatalog | ||||
Diff against target: |
254 lines (+148/-18) 7 files modified
django_project/config/main.cfg (+10/-3) setup.py (+3/-0) src/webcatalog/schema.py (+8/-3) src/webcatalog/templates/500.html (+0/-3) src/webcatalog/tests/__init__.py (+1/-0) src/webcatalog/tests/test_oops_wsgi_config.py (+110/-0) src/webcatalog/wsgi.py (+16/-9) |
||||
To merge this branch: | bzr merge lp:~michael.nelson/ubuntu-webcatalog/render_404_rather_than_raise_exception | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
James Westby (community) | Approve | ||
Review via email: mp+121016@code.launchpad.net |
Commit message
Use oops-wsgi for errors.
Description of the change
Overview
========
Contrary to the name, this branch adds and configures oops-wsgi for use with ubuntu-webcatalog.
`fab test`
To post a comment you must log in.
68 + 'instance_id': 'dev',
It is preferred not to set instance_id at all.
16 +[oops_ datedir_ publisher]
17 +type = datedir
18 +error_dir = /tmp/uwc-oops/
19 +instance_id = dev
Why override from the default here? Having the oopses in-tree seems
preferable to me anyway.
Other than that this looks excellent.
Thanks,
James