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.

Sign up for GitHub

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

Jump to bottom

Final document verification #484

Closed
2 of 3 tasks
banterCZ opened this issue Nov 1, 2022 · 0 comments
Closed
2 of 3 tasks

Final document verification #484

banterCZ opened this issue Nov 1, 2022 · 0 comments
Assignees
@banterCZ

Comments

@banterCZ
Copy link
Member

banterCZ commented Nov 1, 2022 •
edited
Loading

Single verification ID common to all documents should be send to Onboarding Provider client evaluation. uploadSessionId is most probably useless.

logs

DocumentProcessingService  : Verify document a92c911d-0c7f-41b7-b504-2998e449f4b2, uploaded-9b8630a2-a827-4c27-905b-...
DocumentProcessingService  : Verify document b8b1bec1-1380-4125-b5c3-bfb2fcba9f61, uploaded-4986d63b-6cd7-4dcd-afcf-...
IdentityVerificationService  : Verify documents 1a5ea207-1608-4d83-aec7-c506317253b0, [uploaded-9b8630a2-a827-4c27-905b-..., uploaded-4986d63b-6cd7-4dcd-afcf-...]
DocumentProcessingService  : Verify document 1cc2fb6a-a4c3-4db8-9aeb-6686681a87f2, uploaded-2cd8c7f9-0e9f-4e38-87c2-...
IdentityVerificationService  : Verify documents 97ae0e17-3260-406a-8e42-5b5e18b5f7d7, [uploaded-2cd8c7f9-0e9f-4e38-87c2-...]
MockOnboardingProvider      : Evaluating client for process ID: d9d4ff30-a5ee-4b00-b2ee-df6d726a62f1, user ID: mockuser_425986498155068775, verification ID: 1a5ea207-1608-4d83-aec7-c506317253b0, identityVerification ID: b056af51-fc49-4c76-b536-a041d5766d34

es_document_verification

id,activation_id,identity_verification_id,type,side,other_side_id,provider_name,status,filename,upload_id,verification_id,photo_id,verification_score,reject_reason,error_detail,original_document_id,used_for_verification,timestamp_created,timestamp_uploaded,timestamp_verified,timestamp_disposed,timestamp_last_updated,error_origin,reject_origin
47fe9ce0-2fef-4314-a6a3-c1d4d13058e2,57ebeba5-d94c-425d-9e3d-b4e1f675119e,b056af51-fc49-4c76-b536-a041d5766d34,DRIVING_LICENSE,FRONT,,mock,ACCEPTED,driving_license_mock_front.png,uploaded-2cd8c7f9-0e9f-4e38-87c2-...,97ae0e17-3260-406a-8e42-5b5e18b5f7d7,extracted-photo-id,,,,,true,2022-11-01 09:36:36.985000,2022-11-01 09:36:36.985000,2022-11-01 09:36:50.028000,,2022-11-01 09:36:50.028000,,
0c9759d2-3c4f-45df-b283-504b62d27d3b,57ebeba5-d94c-425d-9e3d-b4e1f675119e,b056af51-fc49-4c76-b536-a041d5766d34,SELFIE_PHOTO,,,mock,ACCEPTED,selfie_photo.jpg,uploaded-selfie-photo-57ebeba5-d9...,,,,,,,false,2022-11-01 09:37:06.009000,2022-11-01 09:37:06.009000,,,,,
47c06a6c-1b23-4d5d-a20b-308123e1c930,57ebeba5-d94c-425d-9e3d-b4e1f675119e,b056af51-fc49-4c76-b536-a041d5766d34,ID_CARD,FRONT,da24994b-799e-43a8-9227-a631555b14f1,mock,ACCEPTED,id_card_mock_front.png,uploaded-9b8630a2-a827-4c27-905b-...,1a5ea207-1608-4d83-aec7-c506317253b0,extracted-photo-id,,,,,true,2022-11-01 09:36:27.691000,2022-11-01 09:36:27.691000,2022-11-01 09:36:35.007000,,2022-11-01 09:36:35.007000,,
da24994b-799e-43a8-9227-a631555b14f1,57ebeba5-d94c-425d-9e3d-b4e1f675119e,b056af51-fc49-4c76-b536-a041d5766d34,ID_CARD,BACK,47c06a6c-1b23-4d5d-a20b-308123e1c930,mock,ACCEPTED,id_card_mock_back.png,uploaded-4986d63b-6cd7-4dcd-afcf-...,1a5ea207-1608-4d83-aec7-c506317253b0,extracted-photo-id,,,,,true,2022-11-01 09:36:27.691000,2022-11-01 09:36:27.691000,2022-11-01 09:36:35.007000,,2022-11-01 09:36:35.007000,,

Solution

  • set isDocumentVerificationOnSubmitEnabled=false
  • cross validate both documents via api/investigate/samples and send this investigation ID to client evaluation
  • remove query parameter uploadSessionId
@banterCZ banterCZ self-assigned this Nov 1, 2022
banterCZ added a commit that referenced this issue Nov 1, 2022
@banterCZ
Fix #484: Consolidate ZenId verification ID
ff543a2 
Filter document verification at getVerificationId
banterCZ added a commit that referenced this issue Nov 1, 2022
@banterCZ
Fix #484: Consolidate ZenId verification ID
29b8458 
Remove query parameter uploadSessionID
banterCZ added a commit that referenced this issue Nov 1, 2022
@banterCZ
Fix #484: Consolidate ZenId verification ID
5dcdf3c 
Add logging
banterCZ added a commit that referenced this issue Nov 1, 2022
@banterCZ
Fix #484: Consolidate ZenId verification ID
62e5e1e 
RequiredDocumentTypesCheck does not verify document status anymore
banterCZ added a commit that referenced this issue Nov 1, 2022
@banterCZ
Fix #484: Consolidate ZenId verification ID
ca109f2 
Add RequiredDocumentTypesGuard
banterCZ added a commit that referenced this issue Nov 2, 2022
@banterCZ
Fix #484: Consolidate ZenId verification ID
436aced 
Filter document verification at getVerificationId
banterCZ added a commit that referenced this issue Nov 2, 2022
@banterCZ
Fix #484: Consolidate ZenId verification ID
1405d0b 
Remove query parameter uploadSessionID
banterCZ added a commit that referenced this issue Nov 2, 2022
@banterCZ
Fix #484: Consolidate ZenId verification ID
813591c 
Add logging
banterCZ added a commit that referenced this issue Nov 2, 2022
@banterCZ
Fix #484: Consolidate ZenId verification ID
8111387 
RequiredDocumentTypesCheck does not verify document status anymore
banterCZ added a commit that referenced this issue Nov 2, 2022
@banterCZ
Fix #484: Consolidate ZenId verification ID
27315a0 
Add RequiredDocumentTypesGuard
banterCZ added a commit that referenced this issue Nov 2, 2022
@banterCZ
Remove query parameter uploadSessionID
8cd2f6b 
Issue #484
banterCZ added a commit that referenced this issue Nov 2, 2022
@banterCZ
Set document-verification.verificationOnSubmitEnabled to false
f3b59ac 
Issue #484
This was referenced Nov 2, 2022
Merged
Merged
banterCZ added a commit that referenced this issue Nov 2, 2022
@banterCZ
Remove query parameter uploadSessionID (#488)
b686dfa 
Issue #484
banterCZ added a commit that referenced this issue Nov 2, 2022
@banterCZ
Fix #484: Consolidate ZenId verification ID
f724afb 
- add logging
banterCZ added a commit that referenced this issue Nov 2, 2022
@banterCZ
Set document-verification.verificationOnSubmitEnabled to false (#487)
fcd0a11 
* Set document-verification.verificationOnSubmitEnabled to false

Issue #484
banterCZ added a commit that referenced this issue Nov 3, 2022
@banterCZ
Log document verification
c3d9dee 
Issue #484
@banterCZ banterCZ mentioned this issue Nov 3, 2022
Merged
banterCZ added a commit that referenced this issue Nov 3, 2022
@banterCZ
Log document verification (#492)
979c461 
Issue #484
@banterCZ banterCZ changed the title Consolidate ZenId verification ID Final document verification Nov 4, 2022
@banterCZ banterCZ mentioned this issue Nov 4, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@banterCZ banterCZ

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@banterCZ