[GitHub] [apisix] membphis commented on a change in pull request #2458: [doc]to correct some typo and incorrect examples in getting-started.md and syslog.md

[GitBox]

membphis commented on a change in pull request #2458:
URL: https://github.com/apache/apisix/pull/2458#discussion_r507176656



##########
File path: doc/getting-started.md
##########
@@ -237,7 +237,7 @@ curl http://127.0.0.1:9080/apisix/admin/routes/5 -H 
'X-API-KEY: edd1c9f034335f13
 Now you can invoke the route with the following command:
 
 ```bash
-curl -i -X GET http://127.0.0.1:9080/samplePrefix/get?param1=foo&param2=bar -H 
'apikey: superSecretAPIKey'
+curl -i -X GET "http://127.0.0.1:9080/samplePrefix/get?param1=foo&param2=bar"; 
-H 'apikey: superSecretAPIKey'

Review comment:
       I prefer to use `'` in this case, the same as `-H`.

##########
File path: doc/plugins/syslog.md
##########
@@ -57,25 +57,22 @@ This will provide the ability to send Log data requests as 
JSON objects.
 The following is an example on how to enable the sys-logger for a specific 
route.
 
 ```shell
-curl http://127.0.0.1:9080/apisix/admin/consumers -H 'X-API-KEY: 
edd1c9f034335f136f87ad84b625c8f1' -X PUT -d '
+curl http://127.0.0.1:9080/apisix/admin/routes/1 -H 'X-API-KEY: 
edd1c9f034335f136f87ad84b625c8f1' -X PUT -d '

Review comment:
       nice catch

##########
File path: doc/zh-cn/getting-started.md
##########
@@ -232,7 +232,7 @@ curl http://127.0.0.1:9080/apisix/admin/routes/5 -H 
'X-API-KEY: edd1c9f034335f13
 现在可以使用以下命令调用路由：
 
 ```bash
-curl -i -X GET http://127.0.0.1:9080/samplePrefix/get?param1=foo&param2=bar -H 
'apikey: superSecretAPIKey'
+curl -i -X GET "http://127.0.0.1:9080/samplePrefix/get?param1=foo&param2=bar"; 
-H 'apikey: superSecretAPIKey'

Review comment:
       ditto




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