Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Censor Rules Cannot be Multi-Line #33

Closed
RichardTaylor opened this issue Mar 12, 2011 · 5 comments
Closed

Censor Rules Cannot be Multi-Line #33

RichardTaylor opened this issue Mar 12, 2011 · 5 comments
Milestone

Comments

@RichardTaylor
Copy link

Currently a multi-line censor rule doesn't work.

This means administrators often have to create tens of single-line censor rules to censor large blocks of text. This can be time consuming and annoying.

@bjh21
Copy link

bjh21 commented Mar 24, 2011

My experience is that they can sometimes be multi-line, in that I can replace what look like newlines in a request with spaces in the censor rule and get useful results.

More generally, though, an option for fuzzier matching of long censor rules would make it easier to create them.

@sebbacon
Copy link
Contributor

sebbacon commented Aug 9, 2011

Can you supply an example so I can write a test for this?

@ghost ghost assigned sebbacon Aug 9, 2011
@RichardTaylor
Copy link
Author

As an example: My first attempt at a censor rule at https://secure.mysociety.org/admin/foi/request/show/93647 failed

@sebbacon
Copy link
Contributor

@pepe has made a start on supporting regex-based censor rules, which might be the right solution for this: pepe@befd6de

@RichardTaylor
Copy link
Author

Request 485932 needed ~20 line by line censor rules to remove material. A censor rules based on multi-line text didn't work.

Perhaps this could have been dealt with by a regular expression censor rule?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants