kwin edited a comment on pull request #9:
URL: 
https://github.com/apache/sling-org-apache-sling-xss/pull/9#issuecomment-843982083


   This happens, because the phase of the animal-sniffer-m-p changed in 1.19+ 
(https://github.com/mojohaus/animal-sniffer/commit/6363af4e07240f6240e9cfd4d2e35fecbe6f41ca#diff-600eed5a54fef6581f6ed4ff10c0bb63143255964d046e9ed3ff524337f9846cR64)
 from PROCESS_CLASSES to PROCESS_TEST_CLASSES. This leads to the fact that the 
classes being embedded (i.e. the ones from OWASP) are now checked as well, 
which fails due to the incomplete compile classpath. Just executing 
animal-sniffer-maven-plugin prior to bnd-maven-plugin would work (but only if 
combined with `clean`). The safer approach is to exclude those embedded classes 
from sniffing....


-- 
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]


Reply via email to