newrelic / newrelic-java-agent

The New Relic Java agent
Apache License 2.0
192 stars 140 forks source link

Log IAST related config variables #1877

Closed sdaubin closed 2 weeks ago

sdaubin commented 2 weeks ago

Overview

Customers are seeing this message:

New Relic Security is completely disabled by one of the user provided config `security.enabled`, `security.agent.enabled` or `high_security`. Not loading security capabilities.

But they believe they have configured the agent to enable IAST. This PR logs security.enabled, security.agent.enabled and high_security at FINE so that the config values can be scrutinized.

Related Github Issue

Include a link to the related GitHub issue, if applicable

Testing

The agent includes a suite of tests which should be used to verify your changes don't break existing functionality. These tests will run with Github Actions when a pull request is made. More details on running the tests locally can be found here,

Checks

codecov-commenter commented 2 weeks ago

Codecov Report

Attention: Patch coverage is 0% with 7 lines in your changes are missing coverage. Please review.

Project coverage is 70.68%. Comparing base (29a390c) to head (28d4baf).

Files Patch % Lines
...-agent/src/main/java/com/newrelic/agent/Agent.java 0.00% 7 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1877 +/- ## ============================================ - Coverage 70.68% 70.68% -0.01% - Complexity 9870 9871 +1 ============================================ Files 828 828 Lines 39832 39838 +6 Branches 6064 6064 ============================================ + Hits 28156 28158 +2 - Misses 8949 8953 +4 Partials 2727 2727 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.