AdguardForMac icon indicating copy to clipboard operation
AdguardForMac copied to clipboard

When reporting a website, the pre-filled information does not parse `%20` correctly

Open ghost opened this issue 3 years ago • 2 comments

Steps to reproduce

  1. Add a userscript to AdGuard that includes %20 in the URL, such as https://greasyfork.org/scripts/398340-reddit-dark-theme-sync/code/Reddit%20Dark%20Theme%20Sync.user.js
  2. Report an issue with a website via the Safari extension
  3. Submit the issue
  4. Visit the GitHub issue link, such as https://github.com/AdguardTeam/AdguardFilters/issues/126566

Expected behavior

The full URL is shown and linked

Actual behavior

%20 is replaced with actual spaces and is a broken URL

Screenshot: image

Your environment

  • Safari Version 15.6.1 (17613.3.9.1.16)
  • AdGuard Version 2.8.1.1147 release (CL-1.9.86, DNS-1.7.22)

ghost avatar Aug 19 '22 03:08 ghost

It is now adding + to all the names and URLs where spaces and %20 used to be.

Screen Shot 2022-08-20 at 8 49 39 pm

ghost avatar Aug 20 '22 10:08 ghost

Reproduced on our side, thank you.

Aydinv13 avatar Aug 24 '22 09:08 Aydinv13