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

NR-106006 fix how user attribute is set in servlet-user from previous PR #1265

Merged
merged 4 commits into from
May 12, 2023

Conversation

obenkenobi
Copy link
Contributor

@obenkenobi obenkenobi commented May 11, 2023

Fixes a problem that is missed in the previous PR for the ticket because GHA was not working so tests did not run. The "user" attribute was changed to a custom attribute but I reverted it to be an agent attribute.

@obenkenobi obenkenobi changed the title NR-106006 fix a bug in user attribute from previous PR NR-106006 fix a change in setting user attribute in servlet-user from previous PR May 11, 2023
@obenkenobi obenkenobi changed the title NR-106006 fix a change in setting user attribute in servlet-user from previous PR NR-106006 fix how user attribute is set in servlet-user from previous PR May 11, 2023
@codecov-commenter
Copy link

codecov-commenter commented May 11, 2023

Codecov Report

Merging #1265 (191b6c6) into main (cb11477) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##               main    #1265      +/-   ##
============================================
- Coverage     57.96%   57.95%   -0.02%     
  Complexity     8523     8523              
============================================
  Files           943      943              
  Lines         42260    42264       +4     
  Branches       6150     6150              
============================================
- Hits          24494    24492       -2     
- Misses        15355    15360       +5     
- Partials       2411     2412       +1     
Impacted Files Coverage Δ
...src/main/java/com/newrelic/api/agent/NewRelic.java 0.00% <ø> (ø)
.../newrelic/api/agent/NewRelicApiImplementation.java 22.48% <0.00%> (-0.31%) ⬇️
...src/main/java/com/newrelic/api/agent/NewRelic.java 2.94% <ø> (ø)

... and 3 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Contributor

@jbedell-newrelic jbedell-newrelic left a comment

Choose a reason for hiding this comment

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

Looks like you've got some leftover java.util.logging.Level imports.

@obenkenobi obenkenobi merged commit 94498fd into main May 12, 2023
@obenkenobi obenkenobi deleted the NR-106006-disable-userid-when-high-security branch May 2, 2024 21:29
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.

4 participants