Comments (16)
I started a port to AngleSharp (AngleSharp branch). A large number of tests are not passing yet, though. This is mostly due to AngleSharp/AngleSharp#101.
from htmlsanitizer.
👍
from htmlsanitizer.
I see some discussion at the Github issue of AngleSharp. Do you think they will solve the things that are needed for the replacement?
from htmlsanitizer.
I've just pushed an update to the AngleSharp branch with only around 10 tests still failing. I think most if not all of these only have to be changed so that the expected output matches the actual output.
from htmlsanitizer.
I see that there is a nuget package for this https://www.nuget.org/packages/HtmlSanitizer/3.1.67-beta and that all tests of the AngleSharp branch pass on my machine. Can this be merged back into master yet?
from htmlsanitizer.
I'm planning to do this in a couple of days if no issues pop up during that time. Does the beta work for your use case?
from htmlsanitizer.
Yes it does, and I don't mind depending on a pre release package for the
time being. Take your time :) I'll report any issues here if I get them.
On Thu, Nov 26, 2015 at 3:22 PM, Michael Ganss [email protected]
wrote:
I'm planning to do this in a couple of days if no issues pop up during
that time. Does the beta work for your use case?—
Reply to this email directly or view it on GitHub
#34 (comment)
.
Sent from my regular computer
http://twitter.com/thefringeninja
http://www.thefringeninja.com/
from htmlsanitizer.
Any plans on merging the anglesharp branch to master?
from htmlsanitizer.
Yes, any day now. I want to include #51 and then merge to master.
from htmlsanitizer.
(y) for AngleSharp
from htmlsanitizer.
3.1.76 uses AngleSharp instead of CsQuery.
from htmlsanitizer.
@mganss I realize the train has left the station already on this, but given that AngleSharp seems to be the root cause of a disproportionate number of new issues, are you really gaining anything over using CsQuery? I personally found CsQuery easier to use; does AngleSharp provide functionality not present in CsQuery, or is it just that AngleSharp is actively maintained?
from htmlsanitizer.
It's mostly because AngleSharp is actively maintained. Just yesterday I opened two PRs and got a response within minutes. In contrast, look what happened with #36.
I also found CsQuery easier to use. I don't think that for our use case there is functionality in AngleSharp that's missing from CsQuery. But there's one aspect I have come to appreciate in AngleSharp: It does exactly the same as a browser, even the API is mostly identical, i.e. you can take any HTML snippet and try it in the browser console.
from htmlsanitizer.
Fair enough. Hopefully, once AngleSharp gets to the 1.0 milestone, they'll realize they can't keep introducing breaking changes. Thanks for the response.
from htmlsanitizer.
. Hopefully, once AngleSharp gets to the 1.0 milestone, they'll realize they can't keep introducing breaking changes.
Couldn't agree more!
PS: CSQuery is also not ported to netstandard, and that is of course important for the (near) future.
from htmlsanitizer.
Looks like 1.0 is near for AngleSharp:
from htmlsanitizer.
Related Issues (20)
- Style attribute triggers a RemovingAttribute Event with an empty value HOT 8
- How to convert < into < and similar things using HtmlSanitizer.sanitize call HOT 3
- list-style-type cjk-ideographic is not allowed? HOT 2
- Newer list-style-type values unsupported HOT 1
- Configuration option to bypass changes for security advisory HOT 10
- Sanitize Vue.js scripts HOT 1
- MSO conditional comments HOT 2
- htmlsanitizer is missing NuGet package README file HOT 1
- AngleSharp missing dependency HOT 2
- Url extra escaping HOT 3
- Error on sanitizing simple post without any invalid char. HOT 3
- about slash in background property HOT 1
- Allow outlook conditional comments HOT 1
- Sanitizer removes "px" from the source style attributes when it's "0px" HOT 1
- Properties in @font-face declaration are removed (font-display, mso-generic-font-family, mso-font-alt) HOT 1
- FilterUrl event not raised for relative URLs if baseUrl is used HOT 1
- href's allow inline javascript? HOT 2
- AngleSharp dependency issue in .NET Framework (IIS-hosted WCF service) HOT 9
- RemovingTag and/or RemovingAttribute does not fire for "<BODY ONLOAD=alert('XSS')>" HOT 1
- css attribute white-space is being removed by default HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from htmlsanitizer.