-
Notifications
You must be signed in to change notification settings - Fork 328
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[proposed enhancement] Error: File size (2838487234) is greater than possible Buffer: #820
Comments
Thanks. We'll take a look at the SARIF file from that run and get back to you. |
Hi @aeisenberg , @adityasharad , Did you get a chance to have a look? |
Hi @aeisenberg , @adityasharad , Any news? |
Hi @aeisenberg , @adityasharad , Is Apache OFBiz the only project crossing this issue? |
Hi @JacquesLeRoux, thank you for your patience and apologies for the time it's taken to get back to you. I've had a look at the SARIF file from the Java analysis on your workflow.
Here is an explanation of what I looked for in the SARIF file:
Why are there so many paths and path locations?
How can we make your runs succeed?
I appreciate this is not an ideal solution, but I hope it's enough to keep your code scanning workflows from failing while we think of a more principled solution. Summary of SARIF alerts, with path and path location counts per alert
|
Hi @adityasharad, Thank you for looking at it, I'm already running CodeQL only for js and it was helpful. All is now OK there. As soon as I will get some time (Log4j did not help so far 😏) I'll follow your 3rd recommendation... |
@aeisenberg @adityasharad @JacquesLeRoux thank you for the information provided here. On Keycloak we have the same issue https://github.com/keycloak/keycloak/actions/runs/1865950407. Reading the discussion here helped to identify the issue at least, which is the number of flow paths per alert. It would be nice to have the steps here documented in the troubleshooting guide, or provide an alternative to suppress those alerts. @JacquesLeRoux your project is not the only one facing those issues, at the same time, it seems that we are the only ones with a large Java codebase and running CodeQL :) |
Hi @abstractj, Since then I did not work on that being deeply involved in OFBiz security. It would be great indeed to not have to handle the number of flow paths per alert upstream of CodeQL |
Hi @aeisenberg , @adityasharad ,
As requested here it the workflow run: https://github.com/apache/ofbiz-framework/actions/runs/1470420767
TIA for you help, much appreciated.
The text was updated successfully, but these errors were encountered: