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

[46618] Changing SiS statsd callback so that it uses actual user ial … #10615

Merged
merged 0 commits into from Sep 6, 2022

Conversation

bosawt
Copy link
Contributor

@bosawt bosawt commented Sep 2, 2022

…instead of requested acr value in statsd and logged values

Description of change

Changing Sign in Service callback statsd and log context so that it uses ial value, which represents the actual verification level the user is authenticated with, rather than acr value, which represents the requested verification value for the user

Original issue(s)

department-of-veterans-affairs/va.gov-team#46618

Things to know about this PR

  • Authenticate with sign in service
  • Check for log that should include ial value as a logged field, like this: [SignInService] [V0::SignInController] callback -- { :type => "logingov", :client_id => "mobile", :ial => 2 }

@joeniquette joeniquette merged commit c0b8cc7 into master Sep 6, 2022
@joeniquette joeniquette deleted the 46618_statsd_for_callback_ial branch September 6, 2022 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

4 participants