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

Search DuckDuckGo for text in Tor Window still uses Google #1758

Closed
cndouglas opened this issue Oct 20, 2018 · 5 comments · Fixed by brave/brave-core#757 or brave/brave-core#5407
Closed

Comments

@cndouglas
Copy link
Contributor

Description

If you right-click some text, it gives an option called "Search DuckDuckGo for …". In Private Window with Tor, the search engine is DuckDuckGo by default. However, if you click the menu option, it searches with Google instead. This is very inconvenient because Google is not Tor-friendly.

Steps to Reproduce

  1. Select hamburger menu > New private window with Tor.
  2. Navigate to any website with text (e.g., github.com).
  3. Right-click any text.
  4. Click Search DuckDuckGo for …

Actual result:

The search is with Google.

Expected result:

The search should be with DuckDuckGo.

Reproduces how often:

Always.

Brave version (brave://version info)

Brave | 0.57.1 Chromium: 70.0.3538.67 (Official Build) dev(64-bit)
Revision | 9ab0cfab84ded083718d3a4ff830726efd38869f-refs/branch-heads/3538@{#1002}
OS | Mac OS X

Reproducible on current release:

  • Does it reproduce on brave-browser dev/beta builds? Yes.

Website problems only:

  • Does the issue resolve itself when disabling Brave Shields? No.
  • Is the issue reproducible on the latest version of Chrome? No.

Additional Information

@dnguneratne
Copy link

I'd like to take a crack at this. Any tips on where I should start?

@diracdeltas
Copy link
Member

In b-l we also had this issue with the 'Search for image' option when you right click on an image. If that menu option in fact exists in Tor sessions, it should just be removed.

@rebron rebron added the priority/P2 A bad problem. We might uplift this to the next planned release. label Oct 26, 2018
@rebron rebron modified the milestones: 1.x Backlog, 0.57.x - Dev Oct 26, 2018
@simonhong
Copy link
Member

Similar issue on private window(non tor) - #1037

@bbondy
Copy link
Member

bbondy commented Oct 30, 2018

Updated milestone to 0.58.x since it was merged there.

@LaurenWags
Copy link
Member

LaurenWags commented Dec 11, 2018

Verified passed with

Brave 0.58.11 Chromium: 71.0.3578.80 (Official Build) beta(64-bit)
Revision 2ac50e7249fbd55e6f517a28131605c9fb9fe897-refs/branch-heads/3578@{#860}
OS Mac OS X
  • Verified STR in description

Verification passed on

Brave 0.58.11 Chromium: 71.0.3578.80 (Official Build) beta (64-bit)
Revision 2ac50e7249fbd55e6f517a28131605c9fb9fe897-refs/branch-heads/3578@{#860}
OS Windows 7 x64

used STR from OP

Verification passed on

Brave 0.58.11 Chromium: 71.0.3578.80 (Official Build) beta (64-bit)
Revision 2ac50e7249fbd55e6f517a28131605c9fb9fe897-refs/branch-heads/3578@{#860}
OS Linux

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