Skip to content

LG-11892: Remove Assure ID integration from the backend#10198

Merged
eileen-nava merged 11 commits intomainfrom
em/11892-remove-assure-id-integration
Mar 4, 2024
Merged

LG-11892: Remove Assure ID integration from the backend#10198
eileen-nava merged 11 commits intomainfrom
em/11892-remove-assure-id-integration

Conversation

@eileen-nava
Copy link
Contributor

@eileen-nava eileen-nava commented Mar 1, 2024

🎫 Ticket

LG-11892: Remove Acuant Doc Auth

🛠 Summary of changes

  • Delete logic in doc auth router that sends request to acuant client
  • Delete acuant client
  • Delete acuant config, requests, and responses
  • Delete acuant fixtures
  • Delete test helper methods that only apply to acuant
  • Delete create_document method from mock client that occurs in acuant client, but not in lexis nexis true id client
  • Delete acuant cropping modes, pii from doc, and sensor types
  • Move Acuant::IssuerTypes and Acuant::ResultCodes up a directory level, because both IssuerTypes and ResultCodes were needed for doc auth functions not limited to acuant.
  • Delete logging about acuant vendor status for idv outages, as well as related tests
  • Delete all tests that tested now-deleted code

After this PR is merged, we should delete acuant-specific backend config variables from both application.yml.default and all environments. I will make a follow-up ticket for this work.

📜 Testing Plan

Provide a checklist of steps to confirm the changes.

  • Confirm all automated tests pass.
  • Confirm that you can proceed through identity verification without biometric comparison
  • Confirm that you can proceed through identity verification with biometric comparison

@eileen-nava eileen-nava changed the title WIP: LG-11892: Remove Assure ID integration from the backend LG-11892: Remove Assure ID integration from the backend Mar 1, 2024
@eileen-nava eileen-nava marked this pull request as ready for review March 1, 2024 22:52
@eileen-nava eileen-nava requested review from a team and dawei-nava and removed request for a team March 1, 2024 22:52
Copy link
Contributor

@dawei-nava dawei-nava left a comment

Choose a reason for hiding this comment

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

LGTM.

@eileen-nava eileen-nava self-assigned this Mar 4, 2024
@eileen-nava eileen-nava requested a review from matthinz March 4, 2024 14:11
@eileen-nava eileen-nava merged commit bc32f29 into main Mar 4, 2024
@eileen-nava eileen-nava deleted the em/11892-remove-assure-id-integration branch March 4, 2024 19:31
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.

4 participants