AdguardTeam / AdguardTeam/AdguardBrowserExtension

Space(s) only as custom filter URL value is considered valid (AdGuard Settings)

Aperta
#2,334 3 commenti 0 reazioni 1 assegnatario Rivendicata da @maximtop Vedi su GitHub
Priority: P4
Lingua principale
TypeScript
Stelle
4.4k
Fork
449
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

### ɪꜱꜱᴜᴇ ᴄʜᴀɴɢᴇʟᴏɢ
#### April 20, 2023
- ➕ Added a screenshot in `Steps to reproduce`.
- ➕ Added an animated GIF regarding an alternative possibility in `Expected behavior`.
___
 
##### 💻 Tested and reproducible on both:
- Version `4.1.53` on Firefox `112.0` — Win 10 Pro 22H2
- Version `4.1.55` on Edge `112.0.1722.48` — Win 10 Pro 22H2
 
# 👋 Steps to reproduce

1. Go to `AdGuard Settings` > Filters > Custom > then click on the `Add custom filter` button.
2. [📸 [ꜱᴄʀᴇᴇɴꜱʜᴏᴛ](https://user-images.githubusercontent.com/4764956/233382151-2f73b665-bb10-4590-9106-99e875d22737.png)] Type one space _or more_ in the URL field and click on the `Next` button.

# Actual behavior

It's considered valid! _(Filter will be added with "1 rule" listed in the pre-validation summary.)_

📷 Current result …

 
![firefox_SLbGbPIBCP](https://user-images.githubusercontent.com/4764956/232282054-f029ffe8-e44a-4bb3-950d-d656e8ff7e71.png)

![firefox_hDDm6Dfakz](https://user-images.githubusercontent.com/4764956/232282083-c1f3fe90-16e3-41f3-a134-3bcbea0e763b.png)

# Expected behavior

_Unless_ this is intended for quick testing purposes (noop filter), it should not be I would say. **AdGuard for Windows** doesn't permit that.

📷 Expected result …

 
![firefox_BZMX3z45DU](https://user-images.githubusercontent.com/4764956/232282465-4309aa3b-efe6-493b-aed2-28e20937ea12.png)

Or, just like in the [MV3](https://github.com/AdguardTeam/AdGuardMV3) experimental version, disable the validation button by default and when this situation occurs:

![msedge_6NCzt02FvT](https://user-images.githubusercontent.com/4764956/233377949-fd6033a4-8ee8-4e27-801a-821ee8d5a9ab.gif)
↪️ Makes sense too, if not more.

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.