[PATCH] D112525: [LNT] Add more information to error messages
Pavel Kosov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Oct 26 04:16:18 PDT 2021
kpdev42 added inline comments.
================
Comment at: lnt/server/ui/views.py:814-815
except ValueError:
- return abort(400)
+ return abort(400, "Parameter {} was malformed. {} must be int.int.int" \
+ .format(name, value))
----------------
thopre wrote:
> Is it still the case, I remember a patch that wanted to allow arbitrary strings
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
Repository:
rLNT LNT
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D112525/new/
https://reviews.llvm.org/D112525
More information about the llvm-commits
mailing list