Skip to content

Scrub the error message from JSON parse errors of LexisNexis response bodies#7412

Merged
jmhooper merged 3 commits intomainfrom
jmhooper-scrub-json-parse-error-message
Nov 30, 2022
Merged

Scrub the error message from JSON parse errors of LexisNexis response bodies#7412
jmhooper merged 3 commits intomainfrom
jmhooper-scrub-json-parse-error-message

Conversation

@jmhooper
Copy link
Contributor

@jmhooper jmhooper commented Nov 30, 2022

The LexisNexis response bodies may contain information that we do not wish to be logged or reported to NewRelic (such as PII).

When a JSON parse error is raised it includes the portion of the JSON that resulted in the error. This commit rescues the JSON parse errors that are raised when a LexisNexis response body is being parsed. These error messages may contain sensitive information. As a result a new JSON parse error is raised in their place with a message that does not include the JSON that caused the error.

[skip changelog]

… bodies

The LexisNexis response bodies may contain information that we do not wish to be logged or reported to NewRelic (such as PII).

When a JSON parse error is raised it includes the portion of the JSON that resulted in the error. This commit rescues the JSON parse errors that are raised when a LexisNexis response body is being parsed. These error messages may contain sensitive information. As a result a new JSON parse error is raised in their place with a message that does not include the JSON that caused the error.

[skip changelog]
@jmhooper jmhooper changed the title Scrub the error message from JSON parse errors of LexisNexis response… Scrub the error message from JSON parse errors of LexisNexis response bodies Nov 30, 2022
@jmhooper jmhooper merged commit 0ceb24b into main Nov 30, 2022
@jmhooper jmhooper deleted the jmhooper-scrub-json-parse-error-message branch November 30, 2022 18:04
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.

3 participants