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.

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

Add a fingerprinting exception for veriff.me for uphold #6983

Closed
jumde opened this issue Nov 18, 2019 · 3 comments · Fixed by brave/brave-core#4012
Closed

Add a fingerprinting exception for veriff.me for uphold #6983

jumde opened this issue Nov 18, 2019 · 3 comments · Fixed by brave/brave-core#4012

Comments

@jumde
Copy link
Contributor

jumde commented Nov 18, 2019

Test plan

See brave/brave-core#4012

Description

Uphold needs veriff.me for KYC, veriff.me uses Canvas APIs. Add a fingerprinting exception to allow veriff.me to access these APIs in 3p context.

@btlechowski
Copy link

btlechowski commented Jan 9, 2020

edit: waiting for access to credentials from @jumde .

@LaurenWags
Copy link
Member

LaurenWags commented Jan 9, 2020

Verified passed with

Brave 1.3.87 Chromium: 79.0.3945.117 (Official Build) beta (64-bit)
Revision 04f0a055010adab4484f7497fbfdbf312c307f1d-refs/branch-heads/3945@{#1019}
OS macOS Version 10.14.6 (Build 18G103)
  • Reproduced the issue with 1.0.0 Chromium: 78.0.3904.97

Screen Shot 2020-01-09 at 12 09 25 PM

Screen Shot 2020-01-09 at 12 09 47 PM

Screen Shot 2020-01-09 at 12 10 06 PM

Screen Shot 2020-01-09 at 12 20 19 PM

Screen Shot 2020-01-09 at 12 20 13 PM

Verification passed on

Brave 1.3.87 Chromium: 79.0.3945.117 (Official Build) beta (64-bit)
Revision 04f0a055010adab4484f7497fbfdbf312c307f1d-refs/branch-heads/3945@{#1019}
OS Ubuntu 18.04 LTS

Verified test plan from brave/brave-core#4012

image

Brave 1.3.87 Chromium: 79.0.3945.117 (Official Build) beta (64-bit)
Revision 04f0a055010adab4484f7497fbfdbf312c307f1d-refs/branch-heads/3945@{#1019}
OS Windows 7 Service Pack 1 (Build 7601.24530)

Verified test plan from brave/brave-core#4012

image

@LaurenWags
Copy link
Member

Note - updated release-notes/include to be release-notes/exclude per discussion with @kjozwiak since this was actually included in 1.1.x.

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