[PATCH 06/10] parser: Use full regexps for delegation rules paths

Johannes Berg johannes at sipsolutions.net
Tue Jan 5 19:38:31 AEDT 2016


On Mon, 2016-01-04 at 10:00 +0000, Finucane, Stephen wrote:

> > I agree with your concerns but haven't given them a thought to be honest. 
> > Right now only patchwork admins can changes the rules, but as you mention we 
> > might not trust them.

Frankly, I'm not quite sure of the permissions model, and even what
"admin" means.

I'm "maintainer" of the linux-wireless project on the kernel.org
patchwork, and in that role I think I should be able to change the
auto-delegate settings.
However, the kernel.org server admin might not trust me with arbitrary
regexps.

> Could we use fnmatch instead? This is the suggestion on StackOverflow
> [1] and
> documentation for the function suggests that the grammar is a very
> simple one
> without the possibility for backrefs or other "dangerous" things [2].

I see no problem with that.

johannes


More information about the Patchwork mailing list