Skip to content

Fix #3055#3056

Merged
MkQtS merged 1 commit intov2fly:masterfrom
luochen1990:fix-regexp-issue
Dec 16, 2025
Merged

Fix #3055#3056
MkQtS merged 1 commit intov2fly:masterfrom
luochen1990:fix-regexp-issue

Conversation

@luochen1990
Copy link
Contributor

Accroding to README.md:

Note: Adding new regexp and keyword rules is discouraged because it is easy to use them incorrectly, and proxy software cannot efficiently match these types of rules.

@Meo597
Copy link
Contributor

Meo597 commented Dec 16, 2025

Unless absolutely necessary, avoid using regular expressions.
They are not only prone to errors (this is an example).
But also have terrible performance. Domain rules don't actually have such strict requirements.
domain and keyword are both fast

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants