##// END OF EJS Templates
Journal Should not return 500 errors on failure, rather better is to return bad request error
marcink -
r1175:8c0a57a4 beta
parent child Browse files
Show More
@@ -31,7 +31,7 b' from sqlalchemy.orm import joinedload, m'
31 from webhelpers.paginate import Page
31 from webhelpers.paginate import Page
32 from itertools import groupby
32 from itertools import groupby
33
33
34 from paste.httpexceptions import HTTPInternalServerError
34 from paste.httpexceptions import HTTPBadRequest
35 from pylons import request, tmpl_context as c, response, url
35 from pylons import request, tmpl_context as c, response, url
36 from pylons.i18n.translation import _
36 from pylons.i18n.translation import _
37 from webhelpers.feedgenerator import Atom1Feed, Rss201rev2Feed
37 from webhelpers.feedgenerator import Atom1Feed, Rss201rev2Feed
@@ -132,7 +132,7 b' class JournalController(BaseController):'
132 self.rhodecode_user.user_id)
132 self.rhodecode_user.user_id)
133 return 'ok'
133 return 'ok'
134 except:
134 except:
135 raise HTTPInternalServerError()
135 raise HTTPBadRequest()
136
136
137 repo_id = request.POST.get('follows_repo_id')
137 repo_id = request.POST.get('follows_repo_id')
138 if repo_id:
138 if repo_id:
@@ -141,11 +141,11 b' class JournalController(BaseController):'
141 self.rhodecode_user.user_id)
141 self.rhodecode_user.user_id)
142 return 'ok'
142 return 'ok'
143 except:
143 except:
144 raise HTTPInternalServerError()
144 raise HTTPBadRequest()
145
145
146
146
147 log.debug('token mismatch %s vs %s', cur_token, token)
147 log.debug('token mismatch %s vs %s', cur_token, token)
148 raise HTTPInternalServerError()
148 raise HTTPBadRequest()
149
149
150
150
151
151
General Comments 0
You need to be logged in to leave comments. Login now