Policy: Payload request/response limits fixes.

- Change policy name.
- Fixed some typos.
- Added description fields on jsonschema form.

Signed-off-by: Eloy Coto <eloy.coto@acalustra.com>

gateway/src/apicast/policy/limits/readme.md → gateway/src/apicast/policy/payload_limits/Readme.md

@@ -1,4 +1,4 @@
-# Request/response size limits policy
+# Request/response payload size limits policy
 
 This policy allows users to limit the size of the request and the response.
 

gateway/src/apicast/policy/limits/apicast-policy.json → gateway/src/apicast/policy/payload_limits/apicast-policy.json

@@ -16,6 +16,7 @@
         "request": {
             "type": "integer",
             "title": "The request limit in bytes",
+            "description": "Request limit in bytes, 0 allows all",
             "default": 0,
             "minimum": 0,
             "examples": [
@@ -25,6 +26,7 @@
         "response": {
             "type": "integer",
             "title": "The response limit in bytes",
+            "description": "Response limit in bytes, 0 allows all",
             "default": 0,
             "minimum": 0,
             "examples": [

gateway/src/apicast/policy/payload_limits/init.lua

@@ -0,0 +1 @@
+return require("payload_limit")

gateway/src/apicast/policy/limits/limits.lua → gateway/src/apicast/policy/payload_limits/payload_limit.lua

@@ -1,7 +1,6 @@
-local _M  = require('apicast.policy').new('Logging Policy', 'builtin')
+local _M  = require('apicast.policy').new('Payload size policy', 'builtin')
 local new = _M.new
 
-local ngx_print = ngx.print
 local ngx_exit = ngx.exit
 local ngx_say = ngx.say
 

t/apicast-policy-limits.t → t/apicast-policy-payload_limits.t

@@ -40,7 +40,7 @@ __DATA__
         ],
         "policy_chain": [
           {
-            "name": "apicast.policy.limits",
+            "name": "apicast.policy.payload_limits",
             "configuration": {
               "request": 0,
               "response": 0
@@ -94,7 +94,7 @@ yay, api backend
         ],
         "policy_chain": [
           {
-            "name": "apicast.policy.limits",
+            "name": "apicast.policy.payload_limits",
             "configuration": {
               "request": 100,
               "response": 0
@@ -128,7 +128,6 @@ Payload Too Large
 
 
 === TEST 3: Request body size smaller than the limit
--- ONLY
 --- backend
   location /transactions/authrep.xml {
     content_by_lua_block {
@@ -150,7 +149,7 @@ Payload Too Large
         ],
         "policy_chain": [
           {
-            "name": "apicast.policy.limits",
+            "name": "apicast.policy.payload_limits",
             "configuration": {
               "request": 10000000,
               "response": 0
@@ -205,7 +204,7 @@ yay, api backend
         ],
         "policy_chain": [
           {
-            "name": "apicast.policy.limits",
+            "name": "apicast.policy.payload_limits",
             "configuration": {
               "request": 0,
               "response": 100
@@ -265,7 +264,7 @@ POST /test
         ],
         "policy_chain": [
           {
-            "name": "apicast.policy.limits",
+            "name": "apicast.policy.payload_limits",
             "configuration": {
               "request": 0,
               "response": 1000000
@@ -295,7 +294,7 @@ POST /test
   }
 
 --- request eval
-"POST /test \n" 
+"POST /test \n"
 --- error_code: 200
 --- no_error_log
 [error]