-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Replace httpbin.org with httpcan.org #2300
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Could you please add more information in the description section? |
|
Great! Could you also take a look at |
test/test_proxy.cc
Outdated
| "/digest-auth/auth/hello/world/SHA-256", | ||
| "/digest-auth/auth/hello/world/SHA-512", | ||
| "/digest-auth/auth-int/hello/world/MD5", | ||
| // "/digest-auth/auth-int/hello/world/MD5", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there any reason why this line is disabled?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I couldn't make auth-int work at all. Even curl fails, so I suppose this is a limitation of httpcan.org.
Sure. It seems to be the same limitation the original |
8cb60fb to
76b48e1
Compare
|
Thanks! I have merged it. |
* Replace httpbin.org with httpcan.org * Fix DigestAuthTest.FromHTTPWatch_Online test
* Replace httpbin.org with httpcan.org * Fix DigestAuthTest.FromHTTPWatch_Online test
* Replace httpbin.org with httpcan.org * Fix DigestAuthTest.FromHTTPWatch_Online test
Because of the recent httpbin.org outages that are failing the proxy tests, I replaced the calls to a rust-based clone called httpcan.org. Another contender is the Go-based httpbingo.org service, but they do not support Brotli or SHA-512 digests. The only difference is in the auth digest, they return the algorithm as part of the response, and I couldn't make auth-int work for some reason.