-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Shield prevents identity verification with webcam when trying to KYC with Uphold #4139
Comments
Please prioritize this issue -- we should create rules that allow Uphold connect to work with shields on. Discussed briefly with @diracdeltas and @evq. Sounds like it is possible. The driver is that Uphold Connect is coming soon for Brave Rewards users -- v 0.68. |
Going to bump this to a |
it worked for me just now with shields on. 0.65.118 Chromium: 75.0.3770.80 (Official Build) (64-bit) MacOS, default shield settings |
@diracdeltas did you try the KYC; with netverify coming in to play (Jumio). Back in Jan when @tomlowenthal and I were playing, we were blocking out KYC. Uphold by itself is not a challenge. The real issue is the KYC part. https://portal.netverify.com/login Thanks. |
@mandar-brave can you or @tomlowenthal post the repro steps to hit that bug? |
@diracdeltas will do; i am trying to see if i can disconnect my ID from my account; i am already KYC'd and Uphold does not show me the verify option. Also assigned issue to Uphold QA to verify if Brave is blocking any of the steps. stay tuned. |
I need to KYC my account either way, so I'll go through the process and document any snags/issues that I run into while using the latest nightly with default shield settings. |
Some quick results/observations:
Going through https://publishers.basicattentiontoken.org and logging into an already created account works without any issues using the default shields settings. I went through 2FA and had no issues logging into Uphold.
Seems like creating a new Uphold account through https://publishers.basicattentiontoken.org is also working. Once you've created a new Publisher account, clicking on Once you fill out the above information and click on Clicking on I managed to fill out all the information and setup my 2FA without any problems using the default shield settings. @mandar-brave seems like it's working (at least on my side) |
However, I'm running into an issue when attempting to KYC with Uphold. Once I've created an account, I go through the KYC process and select "License" as my identification. I accept all the permission prompts from Brave to let Brave us the web cam and run into the following error every time: Once the above fails, clicking on The only data that I ended up providing was my address without ID because the above failed but it seems like Uphold accepted the application even though it's missing information and put it under review. I'm basically stuck now until Uphold asks for more information or locks my account due to incorrect/insufficient information. Retrying the KYC process won't work as you're already in a "Pending" state. @mandar-brave I managed to reproduce the above three times. I think this is a major issue in terms of users being able to KYC without any issues. Has anyone run into the above before? |
@kjozwiak thanks for the detailed repro steps; i edited the issue title accordingly |
Exceptions that we added before (when doing in-product KYC via a modal) can be seen here: Specifically you can check |
i have a fix for this; will add tests then open |
@brave/legacy_qa when testing this issue, please test on new and upgraded profiles. |
I'm still having trouble with this one. @kjozwiak @diracdeltas could one of you take a look to see if you see the same? At the step where I have to upload the front of my drivers license I am getting the below screen (I have confirmed that I am trying to upload a .png file): If I go to shields, I see this: Allowing all device recognition attempts will allow my KYC to proceed. Seems like maybe we need something additional for this issue to be resolved? |
Logged #5800 for #4139 (comment) per discussion with @diracdeltas |
@mandar-brave looks like I'm getting this error |
I think we can send this to Uphold for fixing in terms of error; but reasonable to expect that this is expected in terms of failure |
Verified passed with
Verification passed on
Verified test plan from brave/brave-core#2773 Verification passed on
Verified test plan from brave/brave-core#2773 |
Test plan
See brave/brave-core#2773
Description
I haven't created an Uphold account yet so when I tried clicking "Connect" through the "Your Uphold Wallet" on the Brave Rewards website and sign up from the given link, I couldn't because of Shield. However, when I disable it I'm able to sign up.
Actual result:
Shield doesn't allow signing up
Expected result:
Sign up with no issues
Reproduces how often:
Easily
Brave version (brave://version info)
Brave: 0.62.51 Chromium: 73.0.3683.103 (Official Build) (64-bit)
Revision: e82a658d8159cabbd4938c1660f9bb00b4a82a23-refs/branch-heads/3683@{#902}
OS: Windows 10 OS Build 17763.437
Reproducible on current release:
I have not yet tried
Website problems only:
Disabling Shield resolves the issue
Not reproducible on latest ver of Chrome
Additional Information
Due note that I did not sign up through the Uphold website itself because I tried signing up from there and I can do it easily even with Shield on. This issue only occurs when signing up through the link given by the "Connect".
This only affects signing up not signing in
The text was updated successfully, but these errors were encountered: