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

Sanitize URL post-filter #156

Closed
stevenmoberg opened this issue Jan 17, 2019 · 4 comments
Closed

Sanitize URL post-filter #156

stevenmoberg opened this issue Jan 17, 2019 · 4 comments

Comments

@stevenmoberg
Copy link

Could you add a filtering callback event for SanitizedURL so that they could be filtered to prevent against XSRF or SSRF from valid URI attributes, or mark it as virtual so that it can be overwritten and extended in sub classes.

Or should requests like these come from pull requests?

@mganss mganss closed this as completed in dfd5e9d Jan 18, 2019
@mganss
Copy link
Owner

mganss commented Jan 18, 2019

Good idea, I've added both. This might be useful for other use cases as well. The args for the new event FilterUrl contain the original, unfiltered URL and the processed URL (possibly null if it's deemed unsafe). In the event handler you can set the latter to a different value or null if you want it removed.

@stevenmoberg
Copy link
Author

wow - thanks for the speedy reply

@smitburde
Copy link

@mganss Do you maintain the release notes for the security issues fixed for latest htmlsanitizer?

@mganss
Copy link
Owner

mganss commented Jun 18, 2019

@smitburde Release notes can be found at https:/mganss/HtmlSanitizer/releases (pretty spartan, I know :)

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