Re: Re^2: HTML::Scrubber - Perl extension for scrubbing/sanitizing html (turn "on" off)

Hi, could you explain this? I haven't seen the notation before. Does it mean instead of onunload etc, he would just have 'load' as the list entry?

Comment on Re: Re^2: HTML::Scrubber - Perl extension for scrubbing/sanitizing html (turn "on" off)

Replies are listed 'Best First'.
Re^4: HTML::Scrubber - Perl extension for scrubbing/sanitizing html (turn "on" off) by tye (Sage) on Apr 20, 2003 at 17:06 UTC
He already has an entry `'' => 1, # default rule, allow all attributes` [download] so I was proposing that he add support for a style of option that would allow all attributes that start with "on" to be disallowed. I proposed two possible ways to allow the user to specify such, `"on" => 0` or `qr/^on/i => 0`. The first would require that the code check for "" in options while the second would require checking for references to Regexp objects. Either one would require extra work in the module. But the huge number of attribute names matching on (and that this list appears to be growing over time) means the extra effort might be worthwhile. - tye	[reply] [d/l] [select]

Replies are listed 'Best First'.

Re^4: HTML::Scrubber - Perl extension for scrubbing/sanitizing html (turn "on" off)
by tye (Sage) on Apr 20, 2003 at 17:06 UTC

He already has an entry

    '*' => 1, # default rule, allow all attributes
[download]

I proposed two possible ways to allow the user to specify such, "on*" => 0 or qr/^on/i => 0. The first would require that the code check for "*" in options while the second would require checking for references to Regexp objects.

Either one would require extra work in the module. But the huge number of attribute names matching on* (and that this list appears to be growing over time) means the extra effort might be worthwhile.

tye

[reply]
[d/l]
[select]