Skip to content
This repository has been archived by the owner on May 10, 2024. It is now read-only.

Add query param stripping #7861

Closed
cuba opened this issue Aug 10, 2023 · 1 comment · Fixed by #7862
Closed

Add query param stripping #7861

cuba opened this issue Aug 10, 2023 · 1 comment · Fixed by #7862

Comments

@cuba
Copy link
Contributor

cuba commented Aug 10, 2023

Description:

We need to implement query param stripping in iOS. Most of the logic is found in brave-core

Test plan

  1. Test all links on the following page: https://fmarier.github.io/brave-testing/query-filter.html
  2. Test all links on the following page: https://fmarier.org/query-filter-test.html (It should also pass the tests that claim that "Don't work as expected"). In this page, trackers such as fbclid should be removed on all pages.
  3. Make sure that back and forward works fine as well on some of these links. There shouldn't be any strangenesses with using back and forward buttons while navigating these links.

Additional Information

@Uni-verse
Copy link
Contributor

Uni-verse commented Aug 29, 2023

Verified on iPad (7th Gen) running iPadOS 16.5.1 using version 1.57 (23.8.28.22)

Example Example Example
IMG_1178 IMG_1179 IMG_1182
IMG_1183 IMG_1184 IMG_1185

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

Successfully merging a pull request may close this issue.

3 participants