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.

Sign up for GitHub

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

Jump to bottom

Add logging options for data payloads #31

Merged
merged 10 commits into from
Apr 20, 2023
Merged

Add logging options for data payloads #31

merged 10 commits into from
Apr 20, 2023

Conversation

adrianmo
Copy link
Owner

@adrianmo adrianmo commented Apr 17, 2023 •
edited
Loading

  • Enhance logging for data payloads: logResponseData and logSampleData now support the following configuration options: Always, OnFailure, and Never. Maintains backwards compatibility with the old boolean values.
  • Truncate text payloads larger than 1024 characters.
  • Do not log binary payloads - replace value with [BINARY DATA] placeholder.
  • Add tests to cover new functionality.
  • Fixes Add sampler data and response data for failed transactions alone #29

@adrianmo adrianmo mentioned this pull request Apr 17, 2023
Closed
@adrianmo
Copy link
Owner Author

@ssharmasurender3 can you please review and test the changes here based on what we discussed?

You can download the JAR file from the latest CI build.

@adrianmo adrianmo merged commit eb7d3fa into master Apr 20, 2023
@adrianmo adrianmo deleted the limit-log-data branch April 20, 2023 10:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add sampler data and response data for failed transactions alone
1 participant
@adrianmo