Skip to content

Log LexisNexis errors even on successful transactions#7943

Merged
jmhooper merged 6 commits intomainfrom
jmhooper-log-ln-errors-always
Mar 7, 2023
Merged

Log LexisNexis errors even on successful transactions#7943
jmhooper merged 6 commits intomainfrom
jmhooper-log-ln-errors-always

Conversation

@jmhooper
Copy link
Contributor

@jmhooper jmhooper commented Mar 7, 2023

In the past we have not logged errors from the response on passed LexisNexis transaction. The assumption here was that if the transaciton passed there must not be any errors.

Due to nuances in the configuration LexisNexis transactions can pass if there are tolerable errors or errors that are not deemed risky enough to fail a transaction. These still contain useful information, however.

This commit makes a change to always log the errors in LexisNexis transactions, even if they are successful.

In the past we have not logged errors from the response on passed LexisNexis transaction. The assumption here was that if the transaciton passed there must not be any errors.

Due to nuances in the configuration LexisNexis transacitons can pass if there are tolerable errors or errors that are not deemed risky enough to fail a transaction. These still contain useful information, however.

This commit makes a change to always log the errors in LexisNexis transacitons, even if they are successful.

changelog: Improvements, Error logging, Errors that occur while communicating with LexisNexis Risk Defense Platform services (i.e. InstantVerify and PhoneFinder) are now logged regardless of the verification status of the RDP transaction.
@jmhooper jmhooper marked this pull request as ready for review March 7, 2023 19:46
Copy link
Contributor

@zachmargolis zachmargolis left a comment

Choose a reason for hiding this comment

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

LGTM

jmhooper and others added 3 commits March 7, 2023 15:08
@jmhooper jmhooper merged commit f0f383a into main Mar 7, 2023
@jmhooper jmhooper deleted the jmhooper-log-ln-errors-always branch March 7, 2023 20:57
@jmdembe jmdembe mentioned this pull request Mar 9, 2023
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.

2 participants