Skip to content

LG-15396 Log the initiating SP on verify-by-mail code entry#11713

Merged
jmhooper merged 1 commit intomainfrom
jmhooper-log-initiating-sp-out-of-band
Jan 7, 2025
Merged

LG-15396 Log the initiating SP on verify-by-mail code entry#11713
jmhooper merged 1 commit intomainfrom
jmhooper-log-initiating-sp-out-of-band

Conversation

@jmhooper
Copy link
Contributor

@jmhooper jmhooper commented Jan 7, 2025

When a user enters a code during GPO there usually is not a service provider present in the SP session. This is because the letter we send instructs users to go directly to Login.gov to enter the code. As a result it is difficult to collect verification events for service providers in the out-of-band case since we cannot group events by properties.service_provider.

This commit adds the initiating service provider to the logged analytics event when a user enters a GPO code. The issuer is already logged on in-person and fraud review so this change allows us to group all out-of-band verification events by service provider.

When a user enters a code during GPO there usually is not a service provider present in the SP session. This is because the letter we send instructs users to go directly to Login.gov to enter the code. As a result it is difficult to collect verification events for service providers in the out-of-band case since we cannot group events by `properties.service_provider`.

This commit adds the initiating service provider to the logged analytics event when a user enters a GPO code. The issuer is already logged on in-person and fraud review so this change allows us to group all out-of-band verification events by service provider.

changelog: Internal, Analytics, Initiating SP issuer is logged on verify-by-mail code entry
@jmhooper jmhooper requested a review from a team January 7, 2025 14:52
@jmhooper jmhooper merged commit 91f10b1 into main Jan 7, 2025
@jmhooper jmhooper deleted the jmhooper-log-initiating-sp-out-of-band branch January 7, 2025 21:40
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