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

[#9631] Limit length of errorMessage for better stability #10413

Merged
merged 1 commit into from
Oct 16, 2023

Conversation

intr3p1d
Copy link
Contributor

No description provided.

@intr3p1d intr3p1d added this to the 2.6.0 milestone Oct 16, 2023
@intr3p1d intr3p1d self-assigned this Oct 16, 2023
@intr3p1d intr3p1d linked an issue Oct 16, 2023 that may be closed by this pull request
@codecov
Copy link

codecov bot commented Oct 16, 2023

Codecov Report

Attention: 4 lines in your changes are missing coverage. Please review.

❗ No coverage uploaded for pull request base (master@333212d). Click here to learn what that means.
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##             master   #10413   +/-   ##
=========================================
  Coverage          ?   38.15%           
  Complexity        ?    11774           
=========================================
  Files             ?     3725           
  Lines             ?    93858           
  Branches          ?    10241           
=========================================
  Hits              ?    35808           
  Misses            ?    55043           
  Partials          ?     3007           
Files Coverage Δ
...iler/context/exception/model/ExceptionWrapper.java 44.11% <100.00%> (ø)
...ntext/exception/model/ExceptionWrapperFactory.java 70.58% <100.00%> (ø)
...xt/monitor/config/DefaultExceptionTraceConfig.java 66.66% <50.00%> (ø)
...der/exception/ExceptionWrapperFactoryProvider.java 0.00% <0.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@intr3p1d intr3p1d force-pushed the limit_error_message branch from 5fbbcd3 to bd63c9e Compare October 16, 2023 07:51
@intr3p1d intr3p1d merged commit 30fec4e into pinpoint-apm:master Oct 16, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Supports recording Exceptions and their statistics
1 participant