Open kouros51 opened 4 years ago
Hi @kouros51 ,
You might be right. Some options have never been tested afaik, and it could just send it to /dev/null. I will try to look at this if I can to fix it but no guarantee Im sorry. If you have already the fix ou if you can make it, do the PR I will be happy to take a look at this
Hi @kouros51
I have made a PR to fix this => #44
It's a hack, it add automatically an exclude to */.jspf => I use it in my project too
let me know if it's ok for you
Note : I think it's the same kind of problem of #17
First of all thank you for this updated version.
We are having a legacy stack with a web-app that is based on JSPs and, using this plugin, we want to validate the JSPs pages compilation before going to production.
The problem is that we found that setting
ignoreJspFragmentErrors
totrue
and usingjspf
as fragment file doe not work and the plugin is scanning them anyway.We found in the code of the plugin that the variable
ignoreJspFragmentErrors
is used only in the logger and never passed along to the JSP compiler as shown in the snippetfurthermore, that flag is not used in any of the underlining validation in the code. The only alternative solution, that we found, is to exclude those files from the scan.
We were wondering if our analysis is valid and raise this bug if it's the case. If it's not valid can you please provide us an example on how to use that flag with JSP fragment files.
Thanks.