789334 - [socorro-crashstats] refactor URL handling in view code

Status: RESOLVED FIXED

(Socorro :: Webapp, task)

Description

[Robert Helmer [:rhelmer]]

We need to refactor the view code so that there are as few entry and validation points for input data as possible.

The current views (in views.py) are a bit of a mess since the old URL handling was very inconsistent, each view had to handle the different styles. When bug 788057 lands, we'll be able to make this consistent across all views (except for /daily and /query, which use extensive HTTP params)

An example of a place where we could further centralize is the set_base_data decorator in views.py

Getting this right is not only important for correctness, but also for security. The fewer places we have to deal with input data from the wild, the better.

Comment 1

[Peter Bengtsson [:peterbe]]

Sorry. Bug is too open-ended to keep open. The views code has significantly changed over the last couple of months.