You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@apisix.apache.org by GitBox <gi...@apache.org> on 2022/01/03 15:39:12 UTC

[GitHub] [apisix] bisakhmondal commented on a change in pull request #5997: docs: list APISIX variable in a separate page

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: notifications-unsubscribe@apisix.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org