newrelic / newrelic-java-agent

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

Log IAST disabled at info instead of warn #1878

Closed sdaubin closed 2 weeks ago

sdaubin commented 2 weeks ago

Overview

WARN log level should be used sparingly. The agent is currently warning when IAST is disabled, but that's the common case and it's not a problem. That message should be logged at info level. INFO is the default log level and customers should have no problem spotting the message at that level.

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 1 lines in your changes are missing coverage. Please review.

Project coverage is 70.68%. Comparing base (08be4cb) to head (88c8d97).

Files Patch % Lines
...-agent/src/main/java/com/newrelic/agent/Agent.java 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1878 +/- ## ========================================= Coverage 70.67% 70.68% Complexity 9871 9871 ========================================= Files 828 828 Lines 39838 39838 Branches 6064 6064 ========================================= + Hits 28157 28158 +1 + Misses 8956 8954 -2 - Partials 2725 2726 +1 ```

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