-
Notifications
You must be signed in to change notification settings - Fork 1
Add a list of Windows apps with known compatibility issues #8
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
base: master
Are you sure you want to change the base?
Conversation
ameshkov
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't like this format, check how it's done in Android. We may need structured data in the future.
You should also add the description to README
…t decided on yet.
…ner.exe to routing_exclusions.json
…ting_exclusions.json
| ## What AdGuard applications use these filtering lists? | ||
|
|
||
| Currently, they apply to AdGuard for Android. | ||
| Currently, they apply to AdGuard for Android and AdGuard for Windows. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
для макоси и клишных блокеров не актуально?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Пока нет, так как в них еще не реализован System wide filtering
…ing_exclusions.json
…ing_exclusions.json
…ing_exclusions.json
…uardTeam/AdguardForWindows#5721 to routing_exclusions.json
…ing_exclusions.json
…ing_exclusions.json
…ing_exclusions.json
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Adds Windows-specific compatibility configuration lists to this repo so AdGuard for Windows can consume them alongside the existing Android lists.
Changes:
- Added Windows app exclusion lists for routing and filtering behavior.
- Added a Windows browser list used to enable HTTPS filtering by default for specific browsers.
- Updated README to document Windows compatibility configuration files and their JSON schema.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
| windows/routing_exclusions.json | New list of Windows apps whose traffic should not be routed through AdGuard. |
| windows/filtering_exclusions.json | New list of Windows apps whose traffic filtering should be disabled. |
| windows/browsers.json | New list of browsers + install-detection conditions for enabling HTTPS filtering by default. |
| README.md | Documents Windows config files and the common JSON entry structure. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Add a list of Windows apps with known compatibility issues