Skip to content
This repository has been archived by the owner on May 14, 2020. It is now read-only.

Fix tx.allow_html #705

Closed
wants to merge 3 commits into from
Closed

Conversation

dune73
Copy link
Contributor

@dune73 dune73 commented Mar 21, 2017

Fixes #704 for 3.0.1.

@csanders-git
Copy link
Contributor

It sounds a tad awkward, it'd say something a little cleaner like

Rule 941150 checks for html attributes src, style and href.

By default, this is allowed, but you can set the tx.allow_html

variable to 0 in order to forbid the use of said attributes in

the payload.

@dune73
Copy link
Contributor Author

dune73 commented Mar 23, 2017

Sounds better indeed. Fixed.

@dune73 dune73 requested a review from lifeforms April 3, 2017 20:10
@csanders-git csanders-git added this to the CRS v3.0.1 milestone Apr 4, 2017
@dune73
Copy link
Contributor Author

dune73 commented Apr 12, 2017

Closing this in favor of a new PR coming soon.

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

Successfully merging this pull request may close these issues.

3 participants