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

Follow up: default shield prevents uploading image for Uphold KYC #5891

Closed
btlechowski opened this issue Sep 4, 2019 · 3 comments · Fixed by brave/brave-core#3381
Closed

Follow up: default shield prevents uploading image for Uphold KYC #5891

btlechowski opened this issue Sep 4, 2019 · 3 comments · Fixed by brave/brave-core#3381
Assignees
@jumde
Labels
feature/rewards feature/shields/fingerprint The fingerprinting (aka: "device recognition") protection provided in Shields priority/P2 A bad problem. We might uplift this to the next planned release. QA Pass-Linux QA Pass-macOS QA Pass-Win64 QA/Test-Plan-Specified QA/Yes release-notes/exclude
Milestone
1.1.x - Release

Comments

@btlechowski
Copy link

btlechowski commented Sep 4, 2019 •
edited
Loading

Follow up to #5800

Steps to Reproduce

  1. Go to sandbox.uphold.com.
  2. Create an acct and begin KYC.
  3. When you go to upload an image of your driver's license, you will be told your file type is incorrect.

Actual result:

Unable to upload the image
image

Expected result:

Able to upload an image

Reproduces how often:

Easily reproduced

Brave version (brave://version info)

Brave 0.69.123 Chromium: 76.0.3809.132 (Official Build) beta (64-bit)
Revision fd1acc410994a7a68ac25bc77513d443f3130860-refs/branch-heads/3809@{#1035}
OS Ubuntu 18.04 LTS

cc @brave/legacy_qa @jumde @rebron
@btlechowski btlechowski added feature/shields/fingerprint The fingerprinting (aka: "device recognition") protection provided in Shields QA/Yes QA/Test-Plan-Specified labels Sep 4, 2019
@btlechowski btlechowski mentioned this issue Sep 5, 2019
Closed
@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Sep 5, 2019 •
edited
Loading

Edited: I am facing the same issue in Windows - 0.69.124 staging environment. Please note this issue is NOT happening in production, I am able to upload .JPG or PNG file without any issues in Prod environment.

image

@bsclifton
Copy link
Member

I believe @jumde has a fix for this - specifically happening because it's sandbox.uphold.com, I believe

@jumde jumde mentioned this issue Sep 6, 2019
Merged
32 tasks
@NejcZdovc NejcZdovc added the priority/P2 A bad problem. We might uplift this to the next planned release. label Sep 9, 2019
@NejcZdovc NejcZdovc added this to the 0.72.x - Nightly milestone Sep 18, 2019
@btlechowski
Copy link
Author

btlechowski commented Oct 29, 2019 •
edited by GeetaSarvadnya
Loading

Verification passed on

Brave 0.72.110 Chromium: 78.0.3904.70 (Official Build) dev (64-bit)
Revision edb9c9f3de0247fd912a77b7f6cae7447f6d3ad5-refs/branch-heads/3904@{#800}
OS Ubuntu 18.04 LTS

Verified test plan from brave/brave-core#3381

image

Verification passed on

Brave 1.1.1 Chromium: 78.0.3904.97 (Official Build) beta (64-bit)
Revision 021b9028c246d820be17a10e5b393ee90f41375e-refs/branch-heads/3904@{#859}
OS Windows 10 OS Version 1803 (Build 17134.1006)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jumde jumde

Labels
feature/rewards feature/shields/fingerprint The fingerprinting (aka: "device recognition") protection provided in Shields priority/P2 A bad problem. We might uplift this to the next planned release. QA Pass-Linux QA Pass-macOS QA Pass-Win64 QA/Test-Plan-Specified QA/Yes release-notes/exclude
Projects
None yet
Milestone
1.1.x - Release
Development

Successfully merging a pull request may close this issue.

Fix 5891: Update fingerprinting exception for sandbox.uphold.com brave/brave-core
7 participants
@jumde @bsclifton @rebron @NejcZdovc @LaurenWags @btlechowski @GeetaSarvadnya