-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
add curl proxy support. #888
Conversation
…he rel documentation during packaging
should we name it curlproxy? these config items could also be used with other request to the outside. Not sure which that would be, but why start with limiting it to curl? |
Thanks Bart for the hint. This also switches "==" to "<>" and not the code actually works ;-)
@bartv2 Good point. I removed the "curl" part. Should be clearer now. |
@DeepDiver1975 @butonic @schiesbn @blizzz @bartv2 Could you review please? Or someone else? |
Can someone review this please? |
@karlitschek rebase is necessary - THX |
👍 |
Can we have another reviewer please? |
looks good 👍 |
New doc system
add a check and a warning if setlocale is working
Thanks Bart for the hint. This also switches "==" to "<>" and not the code actually works ;-)
…o add_curl_proxy
Fixes #504
#504
Please review.
Sorry for the second pull request. You shouldn't use git when you are in a hurry ;-)