Performance got degraded after upgrade from 1.5.7 version to 1.5.8 for the SAXScanner. about antisamy

[davewichers]

I think we need @davidbarbrowatwork to review your requested change because I believe it was his changes to 1.5.8 that affected the code you are trying to fix. Can you try to find/nudge him to get him to review/approve your requested change?

[dharmendra027]

Hi @davidbarbrowatwork, I found the performance is down by almost 50% in the 1.5.8 antisamy version comapre to 1.5.7.
I have debugged the issue and raised a pull request for it on @davewichers repository.
Can you please review it once as it is urgent for me.

[dharmendra027]

Hi @davewichers , can you please try to connect to @davidbarbrowatwork to review this PR, as I couldn't get any response from.

[davewichers]

I tried as well, but I can try again. You can find him and try on Twitter. Dave

…

On Feb 4, 2020, at 1:58 AM, Dharmendra Kumar ***@***.***> wrote:  Hi @davewichers , can you please try to connect to @davidbarbrowatwork to review this PR, as I couldn't get any response from. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.

[dharmendra027]

Hi @davewichers, if you have twitter id of @davidbarbrowatwork can you please share.

Thanks

[davewichers]

I think it is @ dbarbrow.

[dharmendra027]

Hi @davewichers,
Is the master branch of this repo is 1.5.8? I found the pom.xml of master master branch have project version 1.5.8 and could not find any specific branch for 1.5.8.

[davewichers]

This is the 1.5.8 release: https://github.com/nahsra/antisamy/releases and I believe there is a checkin tag for that release.
I've created a 1.5.9 branch, so I suggest you make any suggested changes to that branch. Hopefully you can add test cases that show the slowdown you are talking about, but correct behavior otherwise. And then when we add your requested change we can see a significant performance improvement but the same correct functionality verified by the test case.

[dharmendra027]

Hi @davewichers, Change that i have made was present in the branches below 1.5.8.

For example

• In version 1.5.4: line number 131 of https://github.com/nahsra/antisamy/blob/1.5.4/src/main/java/org/owasp/validator/html/scan/AntiSamySAXScanner.java
• Version 1.5.5: line number 131 of https://github.com/nahsra/antisamy/blob/1.5.5/src/main/java/org/owasp/validator/html/scan/AntiSamySAXScanner.java
• Version 1.5.6: line number 131 of https://github.com/nahsra/antisamy/blob/1.5.6/src/main/java/org/owasp/validator/html/scan/AntiSamySAXScanner.java
• Version 1.5.7: line number 131 of https://github.com/nahsra/antisamy/blob/1.5.7/src/main/java/org/owasp/validator/html/scan/AntiSamySAXScanner.java

And I got an instruction to use 1.5.8 version of the Antisamy.So I request to please consider it for version 1.5.8.
Thanks

[davewichers]

The requested line has been added, and this fix was just released in version v1.5.9 of AntiSamy, along with a few dependency upgrades.