mosip / inji-wallet

MIT License
24 stars 83 forks source link

A resident should be able to prove their presence using the verifier's phone #55

Closed rakhimosip closed 1 year ago

rakhimosip commented 2 years ago

Problem statement:

A resident should be able to prove presence using the verifier’s phone.

Prerequisite:

Proposed solution:

NOTE: Creating a Verifiable Presentation when a successful auth happens has been removed from the scope of this feature.

Process flow: image

Mock UI When resident scans QR code on relying party's device, they see the following screen image

Open questions

Tech Notes:

Re-use already available SDKs for selfie auth

============================ SUBTASKS

rakhimosip commented 2 years ago

As per the discussion today with the team, updated the requirement-> Creating a Verifiable Presentation when a successful auth happens has been removed from the scope of this feature.

CC @kyanthony @pmigueld @Anushree09-N @santhoshsunder

rakhimosip commented 1 year ago

As per the discussion, yesterday with the team updated the requirement -> When the resident clicks on the consent checkbox, the 'Verify presence and share' button will be disabled as the face-auth will be performed on the verifier's app (requesting device)

CC @kyanthony @pmigueld @Anushree09-N @santhoshsunder

jannahadlaon commented 1 year ago

Tested on: Android: QA4_io.mosip.residentapp-0.4.1-rc1-newlogic_20221123_1723.apk iOS: inji 0.4.1 (3) (TestFlight)

WORKS AS EXPECTED @pmigueld @kyanthony @danicaerediano

STEPS

  1. Share ID
  2. On the sharing device click on the consent box
  3. Verify that Accept request and verify button is disabled
  4. Add reason for sharing
  5. Select ID
  6. Click on Accept request and choose ID
  7. Click on Accept request and verify button on the Receiving device
  8. Verify that after clicking the button, camera pop up
  9. Capture photo of yourself iOS build: Should show success message (since Face-matching in iOS is mocked) Android: Error should appear
  10. Click on Try again in the error message
  11. Capture photo of the person in the ID
  12. Verify that Success message is shown
  13. Verify that transaction was logged correctly in History

Screenshot iOS Sharing device: https://user-images.githubusercontent.com/102940764/203693859-e8da96fe-0fc3-40b4-8351-8dd77d4fb4b5.mov Receiving device: IMG_0062 IMG_0063 IMG_0064

Android Error message: Screenshot_20221124-123755_MOSIP Resident App Sharing device: Uploading XRecorder_24112022_122604.mp4… Screenshot_20221124-122711_MOSIP Resident App Receiving device: Screenshot_20221124-122626_MOSIP Resident App Screenshot_20221124-122641_MOSIP Resident App Screenshot_20221124-122703_MOSIP Resident App

rakhimosip commented 1 year ago

@pmigueld @Anushree09-N based on our discussion today, I have updated the story with the following note:

jannahadlaon commented 1 year ago

Tested on Android: QA4_io.mosip.residentapp-0.4.1-rc3-newlogic_20221205_0904.apk iOS: Inji 0.4.1 (4)

WORKS AS EXPECTED @pmigueld @danicaerediano @Anushree09-N @kyanthony @rakhimosip @Sujithbn

NOTE: For iOS, Face authentication is mocked so we will always have a successful result for face verification

Test devices: iPhone SE - iOS 15.4 iPhone 11 - iOS 16.1.1 Samsung Galaxy A23 - Android 12 Samsung Galaxy A20s - Android 11

Screenshots Android Screenshot_20221205-165013_MOSIP Resident App

iOS IMG_3126

Raginikrishnamurthy commented 1 year ago

Verified the above story with Google near by protocol where consent feature was available. This feature is excluded in ble protocal, hence is obsolete and closing it.