Skip to content

#1276 Feature Request: NeuVector pod logs in JSON format - #1320

Merged
lsongsuse merged 3 commits into
neuvector:mainfrom
xingzhang-suse:json-log
Sep 25, 2026
Merged

lsongsuse merged 3 commits into
neuvector:mainfrom
xingzhang-suse:json-log

Conversation

@xingzhang-suse

Copy link
Copy Markdown
Collaborator

Description

Fix #1276

Added the log4j reconfiguration for the log format in the scala for runtime switch.
The environment variable is the same as controller's - NV_LOG_FORMAT

Test

Set the environment variable NV_LOG_FORMAT as json | string | plain. Start the web server and observe the manager log format

Additional Information

Tradeoff

Potential improvement

@xingzhang-suse
xingzhang-suse requested a review from a team as a code owner September 24, 2026 00:17
Comment thread build.sbt Outdated
"javax.xml.bind" % "jaxb-api" % "2.3.1",
"com.sun.xml.ws" % "jaxws-ri" % "4.0.3",
"javax.xml.soap" % "javax.xml.soap-api" % "1.4.0",
"log4j" % "log4j" % "1.2.17",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The unmaintained log4j:1.2.17 will trigger critical CVE alerts during container image scanning. Updating the dependency to a secure alternative (reload4j or standard Logback) is recommended.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is upgrade into 2.16.1.

Comment thread admin/src/main/scala/com/neu/core/Core.scala Outdated
@xingzhang-suse

Copy link
Copy Markdown
Collaborator Author

There are some sbt build warnings as well

Comment thread admin/src/main/scala/com/neu/core/Core.scala
@lsongsuse
lsongsuse merged commit 1bc453d into neuvector:main Sep 25, 2026
3 checks passed
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

Successfully merging this pull request may close these issues.

Feature Request: NeuVector pod logs in JSON format

3 participants