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

Native Server: Logging not configurable #67

Open
ryanemerson opened this issue May 13, 2020 · 3 comments
Open

Native Server: Logging not configurable #67

ryanemerson opened this issue May 13, 2020 · 3 comments
Labels
bug Something isn't working config-generator Issue involves changes to the config-generator artifact infinispan/server-native

Comments

@ryanemerson
Copy link
Collaborator

The quarkus native binary does not consume a lo4j2.xml file, instead logging must be configured via properties.

The config-generator should be updated so that it can automatically generate a file containing the required quarkus properties for the logging config specified in the server.yaml. This can then be combined with a .env file containing the other runtme properties, such as -Dquarkus.infinispan-server.config-file to launch the native binary.

@ryanemerson ryanemerson added bug Something isn't working infinispan/server-native config-generator Issue involves changes to the config-generator artifact labels May 13, 2020
@calohmn
Copy link

calohmn commented Dec 16, 2021

@ryanemerson Looking at the fact that log4j2.xml is not used and that configuration has to be done via quarkus properties:
Is it that logging isn't done via the log4j-core module logger implementation here and hence the native server image isn't affected by CVE-2021-44228?

@tristantarrant
Copy link
Member

@calohmn the CVE can only work with a JVM because it requires the ability to execute bytecode. A native image cannot do that.

@calohmn
Copy link

calohmn commented Dec 16, 2021

@tristantarrant Yes, that part is clear, so the full exploits won't work.
However, I was thinking about whether the log4j lookup feature, doing variable expansion in a log message (e.g. ${hostname}) and doing the LDAP server requests alone would work. This in itself could be a problem, albeit a lesser one, for sure.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working config-generator Issue involves changes to the config-generator artifact infinispan/server-native
Projects
None yet
Development

No branches or pull requests

3 participants