Add some useful information to HTTP errors with codes 400 and 404 to make it easier to understand which argument is missing or incorrect.
Details
Diff Detail
- Repository
- rLNT LNT
Event Timeline
lnt/server/ui/views.py | ||
---|---|---|
817–819 | Is it still the case, I remember a patch that wanted to allow arbitrary strings |
lnt/server/ui/views.py | ||
---|---|---|
817–819 | AFAIK, yes, URL is expected to contain something like ?plot.0=1.2.3 . If you can point to a patch, which allows arbitrary string, it would be useful and I'll change this part |
lnt/server/ui/views.py | ||
---|---|---|
817–819 | It is just a copy of the line 1716. The same is in the line 855. |
lnt/server/ui/views.py | ||
---|---|---|
817–819 | I was thinking of https://reviews.llvm.org/D109577, I don't know if it's related to the value here. |
lnt/server/ui/views.py | ||
---|---|---|
817–819 | They are separate. This one is the data lookup used internally to resolve a particular data set; where as, the one int the review is the order format, which is used for x-axis and change-over-time calculations. |
lnt/server/ui/views.py | ||
---|---|---|
817–819 | LGTM too then |
Is it still the case, I remember a patch that wanted to allow arbitrary strings