Make admin activity widget date range parsing a little bit more robust.

Review Request #5101 — Created Dec. 11, 2013 and submitted — Latest diff uploaded

Information

Review Board
release-1.7.x

Reviewers

Make admin activity widget date range parsing a little bit more robust.

It was possible to cause the activity widget in the admin to generate requests
to the server that would generate a 500 response. The issue here was that we
didn't properly sanitize dates unless both the start and end range were
provided.

I've changed it to convert the request parameters to datetimes individually
instead of if they're both necessary, and changed the if/elif/etc block to
verify that the necessary data was provided in request.GET before trying to use
it.

Checked out the admin UI a bit.

    Loading...