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

Tracking through external links in code analyzer popups #2029

Open
1 of 2 tasks
pimlie opened this issue Sep 14, 2024 · 3 comments
Open
1 of 2 tasks

Tracking through external links in code analyzer popups #2029

pimlie opened this issue Sep 14, 2024 · 3 comments
Labels
extension something awry in an extension

Comments

@pimlie
Copy link

pimlie commented Sep 14, 2024

Describe the bug

As the readme states that Telemetry is disabled, it seems to be undesired behaviour as opening links with tracking parameters means a form of telemetry still exists.

Please confirm that this problem is VSCodium-specific

  • This bug doesn't happen if I use Microsoft's Visual Studio Code. It only happens in VSCodium.

This question is N/A, as VS Code does not state that Telemetry is disabled :)

Please confirm that the issue/resolution isn't already documented

To Reproduce

The one code analyzer I know that does this is Dart, but there might be others.

Steps to reproduce the behavior:

  1. Add dart-code.dart-code extension
  2. Create a new file test.dart
  3. Copy paste:
void main() {
  runApp();
}
  1. The runApp call should be red underlined as it's undefined
    image
  2. Hover with your mouse over the error to trigger the code analysis popup
    image
  3. Click on undefined_function
  4. You will then get the popup that asks Do you want VSCodium to open the external website, you can configure Trusted Domains from that popup but it does not mention anything with regards to tracking links. If you click Open then it will open the link: https://dart.dev/tools/diagnostic-messages?utm_source=dartdev&utm_medium=redir&utm_id=diagcode&utm_content=undefined_function#undefined_function

Expected behavior

That VSCodium would remove the utm_* parameters. Basically similar functionality as Firefox's Copy Link Without Site Tracking
image

Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

  • OS: [e.g. Mac OS, Ubuntu 20.04, etc]
  • Architecture [e.g. x64, ia32, arm64]
  • Version [e.g. 1.33.0]
  • App Manager [e.g. Winget, Homebrew, Snap, AUR, RPM, Nix, ...]
  • Sandboxed [e.g. no, Flatpak, Snap]

Additional context

Although the source of the link is an extension and not VSCodium, I think this issue could be solved within VSCodium's Do you want VSCodium to open the external website? popup

@pimlie pimlie added the bug Something isn't working label Sep 14, 2024
@daiyam
Copy link
Member

daiyam commented Sep 14, 2024

It's a link generated by the Dart extension.

We don't have control over any extensions so they can have telemetry or links with tracking.

@daiyam daiyam added extension something awry in an extension and removed bug Something isn't working labels Sep 14, 2024
@pimlie
Copy link
Author

pimlie commented Sep 14, 2024

Thanks. But the links are intercepted by the Do you want VSCodium to open the external website? popup right? And that popup is from VSCodium not from the extension?

So within that popup it should be possible to strip the link from at least known tracking parameters similar as Firefox does?

@pimlie
Copy link
Author

pimlie commented Sep 15, 2024

So we could add a patch to strip tracking query parameters here: https://github.com/microsoft/vscode/blob/main/src/vs/workbench/contrib/url/browser/trustedDomainsValidator.ts#L80

According to the Firefox Source Docs the list with query params they strip can be found here. But that doesn't seem to be complete, as it doesn't mention utm tags but they are still stripped.

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

No branches or pull requests

2 participants