Mercurial > public > mercurial-scm > hg-stable
comparison mercurial/hgweb/hgweb_mod.py @ 39793:b63dee7bd0d9
global: replace most uses of RevlogError with StorageError (API)
When catching errors in storage, we should be catching
StorageError instead of RevlogError. When throwing errors related
to storage, we shouldn't be using RevlogError unless we know
the error stemmed from revlogs. And we only reliably know that
if we're in revlog.py or are inheriting from a type defined in
revlog.py.
Differential Revision: https://phab.mercurial-scm.org/D4655
author | Gregory Szorc <gregory.szorc@gmail.com> |
---|---|
date | Tue, 18 Sep 2018 16:47:09 -0700 |
parents | 17ca967e9fca |
children | 2cd5f1fac788 |
comparison
equal
deleted
inserted
replaced
39792:cb65d4b7e429 | 39793:b63dee7bd0d9 |
---|---|
433 msg = 'revision not found: %s' % err.name | 433 msg = 'revision not found: %s' % err.name |
434 | 434 |
435 res.status = '404 Not Found' | 435 res.status = '404 Not Found' |
436 res.headers['Content-Type'] = ctype | 436 res.headers['Content-Type'] = ctype |
437 return rctx.sendtemplate('error', error=msg) | 437 return rctx.sendtemplate('error', error=msg) |
438 except (error.RepoError, error.RevlogError) as e: | 438 except (error.RepoError, error.StorageError) as e: |
439 res.status = '500 Internal Server Error' | 439 res.status = '500 Internal Server Error' |
440 res.headers['Content-Type'] = ctype | 440 res.headers['Content-Type'] = ctype |
441 return rctx.sendtemplate('error', error=pycompat.bytestr(e)) | 441 return rctx.sendtemplate('error', error=pycompat.bytestr(e)) |
442 except error.Abort as e: | 442 except error.Abort as e: |
443 res.status = '403 Forbidden' | 443 res.status = '403 Forbidden' |