JacquesLeRoux commented on pull request #425: URL: https://github.com/apache/ofbiz-framework/pull/425#issuecomment-998971560
@bjugl , I'm no longer able to review the js CodeQL analysis (because of npm use). There were no real issues. I saw for instance that they used things like \. that are of no use un js since you actually need \\. for escaping a dot in js. But anyway it was not a security issue, just that it's the same to do \.min than .min. We could have reported the other issues (like 4 or 5) o but it's no longer available. So +1 to push. -- 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. To unsubscribe, e-mail: notifications-unsubscr...@ofbiz.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
