kayx23 commented on code in PR #10984:
URL: https://github.com/apache/apisix/pull/10984#discussion_r1508422210
##########
docs/en/latest/plugins/ua-restriction.md:
##########
@@ -39,11 +39,11 @@ A common scenario is to set crawler rules. `User-Agent` is
the identity of the c
| bypass_missing | boolean | False | false |
| When set to `true`, bypasses the check when the `User-Agent` header is
missing. |
| allowlist | array[string] | False | |
| List of allowed `User-Agent` headers.
|
| denylist | array[string] | False | |
| List of denied `User-Agent` headers.
|
-| message | string | False | "Not allowed" | [1, 1024] |
Message with the reason for denial to be added to the response.
|
Review Comment:
`[1, 2024]` was removed because it was confusing. `message` is a string
type, and yet, the valid value was given in numeric range.
Upon checking the code, the numeric range meant the min and max length:
i think this can just be removed.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
