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/02/20 05:35:36 UTC

[GitHub] [apisix] xuanyanwow opened a new pull request #6385: optimization curl command

xuanyanwow opened a new pull request #6385:
URL: https://github.com/apache/apisix/pull/6385


   In some special env. the test curl command may get the error result.
   ``` sh
   # curl "http://127.0.0.1:9080/apisix/admin/services/" -H 'X-API-KEY: edd1c9f034335f136f87ad84b625c8f1'
   {"error_msg":"failed to check token"}
   curl: (6) Could not resolve host: edd1c9f034335f136f87ad84b625c8f1'
   ```
   
   
   references: https://blog.csdn.net/u011035397/article/details/109452045
   
   ### What this PR does / why we need it:
   <!--- Why is this change required? What problem does it solve? -->
   <!--- If it fixes an open issue, please link to the issue here. -->
   
   ### Pre-submission checklist:
   
   <!--
   Please follow the PR manners:
   1. Use Draft if the PR is not ready to be reviewed
   2. Test is required for the feat/fix PR, unless you have a good reason
   3. Doc is required for the feat PR
   4. Use a new commit to resolve review instead of `push -f`
   5. If you need to resolve merge conflicts after the PR is reviewed, please merge master but do not rebase
   6. Use "request review" to notify the reviewer once you have resolved the review
   7. Only reviewer can click "Resolve conversation" to mark the reviewer's review resolved
   -->
   
   * [ ] Did you explain what problem does this PR solve? Or what new features have been added?
   * [ ] Have you added corresponding test cases?
   * [ ] Have you modified the corresponding document?
   * [ ] Is this PR backward compatible? **If it is not backward compatible, please discuss on the [mailing list](https://github.com/apache/apisix/tree/master#community) first**
   


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



[GitHub] [apisix] xuanyanwow commented on pull request #6385: docs: optimization curl command

Posted by GitBox <gi...@apache.org>.
xuanyanwow commented on pull request #6385:
URL: https://github.com/apache/apisix/pull/6385#issuecomment-1046447920


   Because i test in windows with docker.
   
   Now i can make sure the question by use ` fiddler ` to get the  request info.
   It is because windows'cmd cannot parse single quote ,but it's have a little diffrent .
   
   ``` 
   curl -H 'X: 123'
   ```
   it can not parse ,throw the error `curl: (6) Could not resolve host:...`
   
   ```
   curl -H 'X:123'
   ```
   it will not throw the error ,but it's not a normal http's header.
   


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



[GitHub] [apisix] spacewander commented on pull request #6385: docs: optimization curl command

Posted by GitBox <gi...@apache.org>.
spacewander commented on pull request #6385:
URL: https://github.com/apache/apisix/pull/6385#issuecomment-1046206971


   Please don't trust a piece of very doubtful information copied from someone's blog. It is a waste of both your time and mine.


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



[GitHub] [apisix] kwanhur commented on pull request #6385: docs: optimization curl command

Posted by GitBox <gi...@apache.org>.
kwanhur commented on pull request #6385:
URL: https://github.com/apache/apisix/pull/6385#issuecomment-1046172432


   `curl` examples appear everywhere in `docs`, should be optimized along with all the others.


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



[GitHub] [apisix] spacewander closed pull request #6385: docs: optimization curl command

Posted by GitBox <gi...@apache.org>.
spacewander closed pull request #6385:
URL: https://github.com/apache/apisix/pull/6385


   


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



[GitHub] [apisix] spacewander commented on pull request #6385: docs: optimization curl command

Posted by GitBox <gi...@apache.org>.
spacewander commented on pull request #6385:
URL: https://github.com/apache/apisix/pull/6385#issuecomment-1046214467


   I guess you forgot to paste a quote during copy & paste, so `edd1c9f034335f136f87ad84b625c8f1` is not quoted and is treated as the target.
   
   Please learn the difference between single quotes and double quotes via `man bash`, instead of believing an idea from an unreliable platform.
   
   


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



[GitHub] [apisix] spacewander edited a comment on pull request #6385: docs: optimization curl command

Posted by GitBox <gi...@apache.org>.
spacewander edited a comment on pull request #6385:
URL: https://github.com/apache/apisix/pull/6385#issuecomment-1046206971


   Please don't trust a piece of very doubtful information copied from someone's blog.


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



[GitHub] [apisix] xuanyanwow commented on pull request #6385: docs: optimization curl command

Posted by GitBox <gi...@apache.org>.
xuanyanwow commented on pull request #6385:
URL: https://github.com/apache/apisix/pull/6385#issuecomment-1046404733


   I make sure what i have not forgot the quote, and i try to use -d or other param with single quote, only the -H working error.
   And this question only show in my one computer, another is normal.
   I will try to find more info and test about this question, and then take a discuss here.
   Thank you all.


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