feat(ext-plugin): support to get request body#5600
Conversation
Signed-off-by: tzssangglass <[email protected]>
Signed-off-by: tzssangglass <[email protected]>
|
I'm thinking that this implementation might not be good enough, because usually the user needs to not only read the request body, but also rewrite it. |
| assert(entry:Name() == "x") | ||
| assert(entry:Value() == "z") | ||
| elseif case.get_request_body then | ||
| assert(call_req:Method() == a6_method.POST) |
There was a problem hiding this comment.
Do we need to check the request_body's value?
There was a problem hiding this comment.
I want to validate different content-type, so the value of request body is not fixed.
There was a problem hiding this comment.
actions.result would check the value in request body?
Signed-off-by: tzssangglass <[email protected]>
|
dealy to merge, is: https://github.com/api7/ext-plugin-proto/blob/main/lua/A6/ExtraInfo/ReqBody.lua used for request body? |
ok, after review:https://lists.apache.org/thread/m5g44gtywx05x4s262rn7qg87oxscy0v, the ReqBody in ExtraInfo is used to holds the request body. |
|
I will update this PR. |
Signed-off-by: tzssangglass <[email protected]>
Signed-off-by: tzssangglass <[email protected]>
Signed-off-by: tzssangglass <[email protected]>
3 participants
Signed-off-by: tzssangglass [email protected]
What this PR does / why we need it:
Pre-submission checklist: