Open
Conversation
Use new docker compose, small cleanups
ec6256a to
96008a5
Compare
nicnac228
reviewed
Oct 19, 2024
| @@ -0,0 +1,5 @@ | |||
| #!/bin/bash | |||
|
|
|||
| openssl req -x509 -newkey rsa:4096 -keyout test-key.pem -out test.pem -sha256 -days 3650 -nodes -subj "/C=XX/ST=Test/L=Test/OU=Test/O=Test/CN=localhost" | |||
There was a problem hiding this comment.
3000:80"
nginx:
build: nginx
ports:
- "3001:80"
- "3002:443"
links:
- "json:api"
squid:
build: squid
ports:
- "3003:3128"
links:
- "nginx:api.example.com"
socks:
image: jgaafromnorth/shinysocks
environment:
- LOG_LEVEL=trace
ports:
- "3004:1080"
links:
- "nginx:api.example.com"
| @@ -0,0 +1,5 @@ | |||
| #!/bin/bash | |||
|
|
|||
| openssl req -x509 -newkey rsa:4096 -keyout test-key.pem -out test.pem -sha256 -days 3650 -nodes -subj "/C=XX/ST=Test/L=Test/OU=Test/O=Test/CN=localhost" | |||
There was a problem hiding this comment.
3000:80"
nginx:
build: nginx
ports:
- "3001:80"
- "3002:443"
links:
- "json:api"
squid:
build: squid
ports:
- "3003:3128"
links:
- "nginx:api.example.com"
socks:
image: jgaafromnorth/shinysocks
environment:
- LOG_LEVEL=trace
ports:
- "3004:1080"
links:
- "nginx:api.example.com"
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
If someone interested, I have added preliminary HTTPS proxy support, known as CONNECT proxy tunnel. It can be used to pass https requests through http proxy (this is how most proxies are working today).
New proxy type Request::Proxy::Type::HTTPS is added for CONNECT type proxy.
No proxy authentication support done yet. Sorry.
Example: (see https://github.com/jgaa/restc-cpp/blob/master/doc/Tutorial.md#send-a-request-going-trough-a-http-proxy)
Also I changed TlsSocket class, so we can pass unencrypted data over TLS socket before tls handshake is done. Otherwise we get tls unintialized protocol error (I know, code is imperfect, more ideas are welcome).
Though I haven't done much testing, sequental proxied requests correctly use the same cached connection to proxy.
If we send proxied requests asyncronously in parallel, they will open new proxy tunnels one for each request.
Testing nginx container is modified so it will serve https requests. Benefit from this, old https tests can be rewritten to request the local nginx instead of the public internet.