This is an automated email from the ASF dual-hosted git repository. kentontaylor pushed a commit to branch kt/discussion-stats in repository https://gitbox.apache.org/repos/asf/allura.git
commit 2f1f1a4816c8847fe066dfb12dfff380d82a0130 Author: Kenton Taylor <[email protected]> AuthorDate: Tue Feb 1 22:01:34 2022 +0000 Prevent discussion stats endpoint from 500ing if hit without parameters --- ForgeDiscussion/forgediscussion/controllers/root.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/ForgeDiscussion/forgediscussion/controllers/root.py b/ForgeDiscussion/forgediscussion/controllers/root.py index 29266fc..b799707 100644 --- a/ForgeDiscussion/forgediscussion/controllers/root.py +++ b/ForgeDiscussion/forgediscussion/controllers/root.py @@ -219,9 +219,9 @@ class RootController(BaseController, DispatchIndex, FeedController): @expose('json:') @validate(dict( - begin=h.DateTimeConverter(if_empty=None, if_invalid=None), - end=h.DateTimeConverter(if_empty=None, if_invalid=None), - )) + begin=h.DateTimeConverter(not_empty=True), + end=h.DateTimeConverter(not_empty=True), + ), error_handler=exc.HTTPBadRequest) def stats_data(self, begin=None, end=None, forum=None, **kw): end = end or date.today() begin = begin or end - timedelta(days=60)
