bdelacretaz commented on issue #5: SLING-8328 - The XSSFilterImpl component cannot start on Jboss 7 deployments URL: https://github.com/apache/sling-org-apache-sling-xss/pull/5#issuecomment-477030164 Do the FactoryFinder classes really need to be duplicated? Instead of having a single class elsewhere from which they inherit. Currently they're bad copies of each other, with the same code AFAICS but extra imports in one of them. Also, it would be nice to briefly explain in SLING-8328 the rationale for this workaround, as it's a bit mysterious (at least to me ;-)
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
