diff options
author | Thomas Bracht Laumann Jespersen <t@laumann.xyz> | 2020-09-28 18:37:07 +0200 |
---|---|---|
committer | Stephen Finucane <stephen@that.guru> | 2020-10-01 15:06:41 +0100 |
commit | b7a6df90802738f729bae144bc618482d9fa6840 (patch) | |
tree | cbf64904bea96cf7b569178de8259f5da5579bc7 /.dockerignore | |
parent | 8f40045ff48235be2552ce2df16ba493b8b83af7 (diff) | |
download | patchwork-b7a6df90802738f729bae144bc618482d9fa6840.tar patchwork-b7a6df90802738f729bae144bc618482d9fa6840.tar.gz |
models: Validate Project.linkname does not contain forward slash
I started by creating a project that contained a forward slash
(importing patches from https://lists.sr.ht/~sircmpwn/sr.ht-dev/) and
it fails to render the "projects" main page.
The specific error reads:
NoReverseMatch at /
Reverse for 'patch-list' with keyword arguments
'{'project_id': 'foo/bar'}' not found. 1 pattern(s) tried:
['project/(?P<project_id>[^/]+)/list/$']
which appears to explicitly disallow forward slashes.
So I think it makes sense to validate that project linkname doesn't
contain forward slahes.
This implementation uses the validate_unicode_slug validator instead of just
rejecting inputs that contain forward slashes.
Signed-off-by: Thomas Bracht Laumann Jespersen <t@laumann.xyz>
Signed-off-by: Stephen Finucane <stephen@that.guru>
Closes: #380
Diffstat (limited to '.dockerignore')
0 files changed, 0 insertions, 0 deletions