membphis commented on a change in pull request #2270: URL: https://github.com/apache/apisix/pull/2270#discussion_r492519914
########## File path: doc/plugins/limit-req.md ########## @@ -104,6 +107,78 @@ Server: APISIX web server This means that the limit req plugin is in effect. +### How to enable on the `consumer` + +To enable the `limit-req` plugin on the consumer, it needs to be used together with the authorization plugin. Here, the key-auth authorization plugin is taken as an example. + +1. Bind the `limit-req` plugin to the consumer + +```shell +curl http://127.0.0.1:9080/apisix/admin/consumers -H 'X-API-KEY: edd1c9f034335f136f87ad84b625c8f1' -X PUT -d ' +{ + "username": "limit_req_consumer_name", + "plugins": { + "key-auth": { + "key": "auth-jack" + }, + "limit-req": { + "rate": 1, + "burst": 1, + "rejected_code": 403, + "key": "consumer_name" + } Review comment: bad indentation ########## File path: doc/plugins/limit-req.md ########## @@ -21,11 +21,14 @@ # Summary -- [**Name**](#name) -- [**Attributes**](#attributes) -- [**How To Enable**](#how-to-enable) -- [**Test Plugin**](#test-plugin) -- [**Disable Plugin**](#disable-plugin) +- [Summary](#summary) + - [Name](#name) + - [Attributes](#attributes) + - [How To Enable](#how-to-enable) + - [Test Plugin](#test-plugin) + - [How to enable on the `consumer`](#how-to-enable-on-the-consumer) + - [Test Plugin](#test-plugin-1) Review comment: [Test Plugin]() [How to enable on the `consumer`]() I think this sorting is better. ########## File path: doc/zh-cn/plugins/limit-req.md ########## @@ -98,6 +98,78 @@ Server: APISIX web server 这就表示 limit req 插件生效了。 +### 如何在`consumer`上启用插件 + +consumer上开启`limit-req`插件，需要与授权插件一起配合使用，这里以key-auth授权插件为例。 + +1、将`limit-req`插件绑定到consumer上 + +```shell +curl http://127.0.0.1:9080/apisix/admin/consumers -H 'X-API-KEY: edd1c9f034335f136f87ad84b625c8f1' -X PUT -d ' +{ + "username": "limit_req_consumer_name", + "plugins": { + "key-auth": { + "key": "auth-jack" + }, + "limit-req": { + "rate": 1, + "burst": 1, + "rejected_code": 403, + "key": "consumer_name" + } Review comment: ditto ########## File path: doc/plugins/limit-req.md ########## @@ -21,11 +21,14 @@ # Summary -- [**Name**](#name) -- [**Attributes**](#attributes) -- [**How To Enable**](#how-to-enable) -- [**Test Plugin**](#test-plugin) -- [**Disable Plugin**](#disable-plugin) +- [Summary](#summary) + - [Name](#name) + - [Attributes](#attributes) + - [How To Enable](#how-to-enable) + - [Test Plugin](#test-plugin) + - [How to enable on the `consumer`](#how-to-enable-on-the-consumer) + - [Test Plugin](#test-plugin-1) Review comment: Important chapters should be first. ########## File path: doc/plugins/limit-req.md ########## @@ -26,8 +26,8 @@ - [Attributes](#attributes) - [How To Enable](#how-to-enable) - [Test Plugin](#test-plugin) - - [How to enable on the `consumer`](#how-to-enable-on-the-consumer) - - [Test Plugin](#test-plugin-1) + - [How to enable on the `consumer`](#how-to-enable-on-the-consumer) + - [Test Plugin](#test-plugin-1) Review comment:  @Firstsawyou ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
