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

brave://wallet alias not shown while looking up an ENS domain or on a suspicious site #8524

Closed
ryanml opened this issue Mar 4, 2020 · 4 comments · Fixed by brave/brave-core#4828

Comments

@ryanml
Copy link
Contributor

ryanml commented Mar 4, 2020

Reproduction steps:

  1. Enable Crypto Wallets via brave://wallet
  2. Visit a site that will trigger phishing detection, a list can be found here: https://github.com/MetaMask/eth-phishing-detect/blob/master/src/hosts.txt
  3. Visit an ENS url, such as http://vitalik.eth

Results:
Screen Shot 2020-03-03 at 8 04 13 PM
Screen Shot 2020-03-03 at 8 04 02 PM

Expected: brave://wallet to be shown in the URL bar

@btlechowski
Copy link

@ryanml For http://vitalik.eth I get:
image

Is this expected behavior?

@ryanml
Copy link
Contributor Author

ryanml commented Mar 24, 2020

@btlechowski yep that's the correct behavior for the final resolution. This fix just made sure that brave://wallet was showing when this was being redirected to. Thanks for checking.

@kjozwiak
Copy link
Member

Quick notes for QA:

@kjozwiak
Copy link
Member

kjozwiak commented Mar 24, 2020

Verification PASSED on macOS 10.15.3 x64 Catalina using the following build:

Brave | 1.7.71 Chromium: 80.0.3987.149 (Official Build) dev (64-bit)
-- | --
Revision | 5f4eb224680e5d7dca88504586e9fd951840cac6-refs/branch-heads/3987_137@{#16}
OS | macOS Version 10.15.3 (Build 19D76)

Screen Shot 2020-03-24 at 12 24 40 PM

Verification passed on

Brave 1.7.71 Chromium: 80.0.3987.149 (Official Build) dev (64-bit)
Revision 5f4eb224680e5d7dca88504586e9fd951840cac6-refs/branch-heads/3987_137@{#16}
OS Ubuntu 18.04 LTS

Verified test plan from the description

image

Verification passed on

Brave 1.7.73 Chromium: 80.0.3987.149 (Official Build) dev (64-bit)
Revision 5f4eb224680e5d7dca88504586e9fd951840cac6-refs/branch-heads/3987_137@{#16}
OS Windows 10 OS Version 1803 (Build 17134.1006)
  • Verified test plan from the description

image

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