-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Automatic Filtering Logic Doesn't Retain Multiple Status Codes with Same Content-Length #738
Comments
Hi, for directory scanning, if there are 5 or 10 files of the same size under the same domain name, then this size will be filtered out. User can customize 5 or 10 or other values. Can this function achieve similar needs ? |
When using AND to connect several filters, if the size is the same, it will
not be filtered out. Only all filters that meet the conditions will be
filtered out.
ayu ***@***.***> 于2023年10月16日周一 09:37写道:
… Hi, for directory scanning, if there are 5 or 10 files of the same size
under the same domain name, then this size will be filtered out. User can
customize 5 or 10 or other values. Can this function achieve similar needs ?
—
Reply to this email directly, view it on GitHub
<#738 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/ALZC53USEPWEZGA4YE2CZ3DX7SFW7AVCNFSM6AAAAAA5YMLL6KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONRTGU4TMMBXGU>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Recently, I encountered an issue. When the automatic filtering is enabled and the status code matched is set to "ALL", if the Content-Length of both 404 and 403 are the same, the 403 won't be retained, which doesn't match my expectations.
After spending some time debugging, I realized that when the automatic filtering is enabled, not all matching filters are added with an "AND" connection. Instead, only the first matching filter is added. I believe this is unexpected behavior. A more reasonable approach would be to add all of them to the "Filters" and use "AND" for filtering decisions.
Below is the modified code I've come up with, just for your reference:
The text was updated successfully, but these errors were encountered: