Skip to content
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

添加异常处理 #193

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

添加异常处理 #193

wants to merge 6 commits into from

Conversation

JoyChou93
Copy link

改了两处。

异常处理

当一个规则如"URIXXX": { "value": "\\.(git|svn)", "operator": "≈" }时,由于没有对URIXXX的处理,所以永远都会return true,所以添加了一个对这个场景的异常处理。

当然,如果通过dashboard不会存在这个问题,不过我最近在使用的时候,改了dashboard,升级后遇到了这个bug,所以提交了这个PR。

!操作符

还有一个问题是,操作符!感叹号,根本用不到这个操作符,但是代码里还有,所以删除了该段代码。

如果这个操作符有特殊用处,麻烦回复一下。我翻了代码没有发现有在使用。

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.

1 participant