bisakhmondal commented on a change in pull request #5997:
URL: https://github.com/apache/apisix/pull/5997#discussion_r777549600



##########
File path: docs/en/latest/plugins/http-logger.md
##########
@@ -95,16 +95,6 @@ hello, world
 
  Note that **the metadata configuration is applied in global scope**, which 
means it will take effect on all Route or Service which use http-logger plugin.
 
-**APISIX Variables**
-
-|   Variable Name  |      Description        | Usage Example  |
-|------------------|-------------------------|----------------|
-| route_id         | id of `route`          | $route_id      |
-| route_name       | name of `route`        | $route_name    |
-| service_id       | id of `service`        | $service_id    |
-| service_name     | name of `service`      | $service_name  |
-| consumer_name    | username of `consumer` | $consumer_name |
-

Review comment:
       As we are deleting the table, I think it would be better to add a 
hyperlink to the newly created `apisix-variable.md` inside the metadata schema 
`log_format` description [L94] (Also same to all the similar updates).




-- 
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]


Reply via email to