diff --git a/app/controllers/admin/reports_controller.rb b/app/controllers/admin/reports_controller.rb index 20b97a6f255..864512c451e 100644 --- a/app/controllers/admin/reports_controller.rb +++ b/app/controllers/admin/reports_controller.rb @@ -88,8 +88,12 @@ class Admin::ReportsController < Admin::AdminController private def parse_params(report_params) - start_date = (report_params[:start_date].present? ? Time.parse(report_params[:start_date]).to_date : 1.days.ago).beginning_of_day - end_date = (report_params[:end_date].present? ? Time.parse(report_params[:end_date]).to_date : start_date + 30.days).end_of_day + begin + start_date = (report_params[:start_date].present? ? Time.parse(report_params[:start_date]).to_date : 1.days.ago).beginning_of_day + end_date = (report_params[:end_date].present? ? Time.parse(report_params[:end_date]).to_date : start_date + 30.days).end_of_day + rescue ArgumentError => e + raise Discourse::InvalidParameters.new(e.message) + end facets = nil if Array === report_params[:facets] diff --git a/spec/requests/admin/reports_controller_spec.rb b/spec/requests/admin/reports_controller_spec.rb index fa1bf1f751e..1f3208746e5 100644 --- a/spec/requests/admin/reports_controller_spec.rb +++ b/spec/requests/admin/reports_controller_spec.rb @@ -47,6 +47,24 @@ describe Admin::ReportsController do expect(JSON.parse(response.body)["reports"][1]["type"]).to eq("not_found") end end + + context "invalid start or end dates" do + it "doesn't return 500 error" do + get "/admin/reports/bulk.json", params: { + reports: { + topics: { limit: 10, start_date: "2015-0-1" } + } + } + expect(response.status).to eq(400) + + get "/admin/reports/bulk.json", params: { + reports: { + topics: { limit: 10, end_date: "2015-0-1" } + } + } + expect(response.status).to eq(400) + end + end end end