Skip to content
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

option to remove problematic classes? #13

Open
4001982248998 opened this issue Dec 16, 2021 · 2 comments
Open

option to remove problematic classes? #13

4001982248998 opened this issue Dec 16, 2021 · 2 comments

Comments

@4001982248998
Copy link

Given that the removal of problematic Java classes from files is trivial, it would very much improve this software if it added an option to do so.

zip -q -d $myfile org/apache/logging/log4j/core/net/JndiLookup.class
zip -q -d $myfile org/apache/logging/log4j/core/net/JndiManager.class
@yunzheng
Copy link
Member

Hi i'm not sure how high demand there is for this, but personally I would prefer to have this tool "read-only" and to only identify the issue on systems.

I want to avoid the tool breaking something on machines. I'm personally also not a big fan of modifying files, but I did see that this was the general advisory of Apache and I understand this could be easier to fix things faster in production rather than waiting for the vendor to release the proper updates.

Besides that, there could also be edge cases for nested Jar files and might potentially break systems because of that, so I think having this an "finder" only tool is better suited unless there is a high demand for it.

@tjyang
Copy link

tjyang commented Dec 16, 2021

  • How about only generate the new patched version in /tmp dir(or other dir) ?
    - and let the app owner do actual swap and check if app ok using patch .jar in /tmp. This way log4j-finder won't got blamed when switching to new version is not working.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants