Skip to content

Render an error when a biometric is requested with a vector of trust#10180

Merged
jmhooper merged 3 commits intomainfrom
jmhooper-biometric-comparison-error
Feb 29, 2024
Merged

Render an error when a biometric is requested with a vector of trust#10180
jmhooper merged 3 commits intomainfrom
jmhooper-biometric-comparison-error

Conversation

@jmhooper
Copy link
Contributor

In #9837 we modified the IdP to render a 406 if biometric comparison was requested but not allowed for the current environment.

This change applied to the query parameter method for requesting biometric comparison which was the only way to request biometric comparison at the time. Since then we have enabled biometric comparison requests using vectors of trust but did not port over the feature for blocking biometric comparison requests in environments where they are not allowed.

This commit applies the feature to requests that use a vector of trust to request biometric comparison.

In #9837 we modified the IdP to render a 406 if biometric comparison was requested but not allowed for the current environment.

This change applied to the query parameter method for requesting biometric comparison which was the only way to request biometric comparison at the time. Since then we have enabled biometric comparison requests using vectors of trust but did not port over the feature for blocking biometric comparison requests in environments where they are not allowed.

This commit applies the feature to requests that use a vector of trust to request biometric comparison.

[skip changelog]
@jmhooper jmhooper requested a review from a team February 28, 2024 21:20
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 jmhooper merged commit fab35dc into main Feb 29, 2024
@jmhooper jmhooper deleted the jmhooper-biometric-comparison-error branch February 29, 2024 14:52
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