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 engine collection still sends requests #2659

Closed
3 tasks
Devocub opened this issue Jan 6, 2024 · 8 comments · Fixed by #2852
Closed
3 tasks

Search engine collection still sends requests #2659

Devocub opened this issue Jan 6, 2024 · 8 comments · Fixed by #2852
Labels

Comments

@Devocub
Copy link

Devocub commented Jan 6, 2024

OS/Platform

Windows

Installed

https://ungoogled-software.github.io/ungoogled-chromium-binaries/

Version

120.0.6099.129

Have you tested that this is not an upstream issue or an issue with your configuration?

  • I have tried reproducing this issue in Chrome and it could not be reproduced there
  • I have tried reproducing this issue in vanilla Chromium and it could not be reproduced there
  • I have tried reproducing this issue in ungoogled-chromium with a new and empty profile using --user-data-dir command line argument and it could not be reproduced there

Description

Request to collect search engine is made when collection is disabled

How to Reproduce?

  1. Enable this chrome://flags/#disable-search-engine-collection
  2. chrome://settings/security Enable "Always use secure connections"
  3. Turn on Wireshark (or alternative)
  4. Go to https://php.net and you should see in Wireshark http request (details below) been made. This request is not visible in browsers DevTools.

Actual behaviour

I believe this line in source code of php.net page is for search engine collection
<link rel="search" type="application/opensearchdescription+xml" href="http://php.net/phpnetimprovedsearch.src" title="Add PHP.net search">
Even though "Always use secure connections" and search engine collection is disabled request still made, and over http (so some security risk).
It is not a bug to be precise because disabling collection doesn't mean disabling requests to this link (and not doing this request = more fingerprinting). And request over http is Chromium bug. But since that was surprising behaviour for me I decided to leave this feedback.

Expected behaviour

No requests.

Relevant log output

No response

Additional context

You can add "Disable serach engine collection requests" flag.
You can report issue about security stuff to Chromium.
You can close this issue immediately.

@Devocub Devocub added the bug label Jan 6, 2024
@PF4Public
Copy link
Contributor

@Devocub Have you attempted to reproduce this issue in the latest available version? Is this still an issue?

@PF4Public PF4Public added the need info Need feedback to proceed label May 11, 2024
@Devocub
Copy link
Author

Devocub commented May 12, 2024

@PF4Public just tried, still reporducible 124.0.6367.201

@PF4Public
Copy link
Contributor

PF4Public commented May 12, 2024

@Ahrotahn IIRC you did test ungoogled-chromium with wireshark and found no suspicious activity. Could this be something you missed?

@networkException networkException removed the need info Need feedback to proceed label May 12, 2024
@networkException
Copy link
Member

Can reproduce

@networkException
Copy link
Member

networkException commented May 13, 2024

I'm actually not sure what codepath the request is taking. Only https://source.chromium.org/chromium/chromium/src/+/main:chrome/renderer/chrome_render_frame_observer.cc;l=259;drc=2246014e44711f62566fcbb784a6926b1ba5b980;bpv=1;bpt=1 looks relevant (which we already disable with the "disable-search-engine-collection" flag)

@rany2
Copy link
Member

rany2 commented May 13, 2024

@networkException https://source.chromium.org/chromium/chromium/src/+/main:chrome/browser/ui/search_engines/search_engine_tab_helper.cc;l=156;drc=317f05fb6ad55aae99ecbf909b233b07198eda3f

@networkException
Copy link
Member

yes but I don't see that being called (over IPC) by anyone else. Oh well, perhaps better to also patch that out

@Ahrotahn
Copy link
Contributor

Yup, y'all are on the right track. The search engine helper shouldn't be loaded at all with the disable-search-engine-collection flag set. I've pushed an update to the PR for 125 to fix this.

Normally I'm checking for connections at startup and for external connections to Google when visiting pages. I wouldn't have thought to check for something like this before since it only makes sense that php.net would connect to php.net.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants