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'