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

Navigating to non-existent/disabled internals page gives chrome:// page error message #10305

Closed
kenherbert opened this issue Jun 16, 2020 · 13 comments · Fixed by brave/brave-core#13244

Comments

@kenherbert
Copy link

Description

When navigating to an internals page that either does not exist or is part of disabled functionality the URI in the address bar uses the correct brave:// scheme but the page error message gives the chrome:// scheme.

Steps to Reproduce

  1. Navigate to brave://test/
  2. Navigate to brave://sync-internals (I am assuming only when sync is disabled?)

Actual result:

Screen Shot 2020-06-16 at 5 44 31 pm

Expected result:

Error message should reference the brave:// scheme in the URI

Reproduces how often:

Always

Brave version (brave://version info)

Brave | 1.10.90 Chromium: 83.0.4103.97 (Official Build) (64-bit)
Revision | 326d148b9655369b86498d9ecca39f63dd2bdd2d-refs/branch-heads/4103@{#657}
OS | macOS Version 10.14.6 (Build 18G87)

Version/Channel Information:

  • Can you reproduce this issue with the current release? Yes
  • Can you reproduce this issue with the beta channel? Untested, but checked recently closed issues for similar issues and could not see it
  • Can you reproduce this issue with the dev channel? As per beta
  • Can you reproduce this issue with the nightly channel? As per beta

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields? Shields unavailable for internals
  • Does the issue resolve itself when disabling Brave Rewards? No
  • Is the issue reproducible on the latest version of Chrome? Yes, but that's part of the problem!

Miscellaneous Information:

@rebron rebron added good first issue priority/P5 Not scheduled. Don't anticipate work on this any time soon. labels Aug 7, 2020
@rebron
Copy link
Collaborator

rebron commented Aug 7, 2020

brave://sync-internals works starting with 1.12.x.

So this issue would be for brave://test brave://*

@swapnilmarkhedkar
Copy link

The issue seems to be only for brave://test/. Since its not assigned to anyone yet, could I work on it?

@bsclifton
Copy link
Member

bsclifton commented Sep 24, 2020

@swapnilmarkhedkar sure - go for it. Let us know if you have any questions 😄

@JuanFML
Copy link

JuanFML commented Nov 28, 2020

Hello everyone, I am new here and I would like to try to solve my first issue, I see someone is working on it, although it has passed some time already, can I give it a try @bsclifton?

@JackMechem
Copy link

@bsclifton Do you mind if i work on this?

@bsclifton
Copy link
Member

@JackMechem sure, let's do it 😄 If nobody is assigned yet and there's not a pull request open, it's fair game

Let me know if you need help setting up or run into an issue

@JackMechem
Copy link

@JackMechem sure, let's do it If nobody is assigned yet and there's not a pull request open, it's fair game

Let me know if you need help setting up or run into an issue

@bsclifton Hey when I run npm run init i get this error:

> git clone  .

fatal: repository 'undefined' does not exist

@bsclifton
Copy link
Member

@JackMechem I think you'll want to check to see if you have the LTS version of Node.js installed. One of the dependencies for the latest Node seems to have a problem with the dash one of the repos our script clones

@JackMechem
Copy link

JackMechem commented Apr 10, 2021

@bsclifton Hey. I have downloaded the LTS version of node and i am still having the same. I tried downgrading node even more and I still had no luck. Could it be an issue with git? If it helps im running Arch Linux

> brave@1.25.3 init
> node ./scripts/init.js

Performing initial checkout of brave-core
Cloning brave-core [undefined] into /home/jack/Documents/con/brave-browser/src/brave...
---------------------------------------------------------------------------------------------
/home/jack/Documents/con/brave-browser/src/brave
> git clone  .

fatal: repository 'undefined' does not exist

~/D/c/brave-browser ❯❯❯ node -v                                                   ✘ 1 master ✱
v10.24.0
~/D/c/brave-browser ❯❯❯                                                                     master ✱

@rahulsnkr
Copy link

rahulsnkr commented Apr 13, 2021

Hey @JackMechem, I was having the same issue and using Node.js v12 LTS worked for me

@fallaciousreasoning
Copy link

I might take a look at this, as it's been quiet for a while, if no one objects.

@stephendonner
Copy link

Verified PASSED using

Brave 1.40.77 Chromium: 102.0.5005.61 (Official Build) beta (x86_64)
Revision 0e59bcc00cc4985ce39ad31c150065f159d95ad3-refs/branch-heads/5005@{#819}
OS macOS Version 11.6.6 (Build 20G624)

Steps:

  1. installed 1.40.77
  2. launched Brave
  3. entered brave://test into the URL bar and pressed return
  4. checked the error messaging

Confirmed brave://test shows, rather than chrome://test in the error messaging.

1.40.77 (brave://test) 1.39.111 (chrome://test)
Screen Shot 2022-05-31 at 12 57 36 PM Screen Shot 2022-05-31 at 12 52 24 PM

@MadhaviSeelam
Copy link

MadhaviSeelam commented May 31, 2022

Verification PASSED using

Brave | 1.40.80 Chromium: 102.0.5005.78 (Official Build) beta (64-bit)
-- | --
Revision | df6dbb5a9fd82af3f567198af2eb5fb4876ef99c-refs/branch-heads/5005_59@{#3}
OS | Windows 11 Version 21H2 (Build 22000.675)

Steps:

  • Installed 1.40.80
  • Launched Brave
  • Entered brave://test into the URL bar and pressed return
  • Checked the error messaging
  • Confirmed brave://test shows, rather than chrome://test in the error messaging.
chrome://test (1.39.111) brave://test (1.40.88)
image image

@rebron rebron changed the title [Desktop] navigating to non-existent/disabled internals page gives chrome:// page error message Navigating to non-existent/disabled internals page gives chrome:// page error message Jun 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

10 participants