Skip to content
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

WIP: add some network diagnostics #589

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

djs55
Copy link
Collaborator

@djs55 djs55 commented Jul 2, 2022

This is to help debug a strange Windows failure:

Add a vpnkit.exe curl http://example.com test program

Add verbose debug logging around TCP connections

djs55 added 10 commits July 13, 2022 13:18
Previously we had
- Endpoint.input_tcp: which handled RST and then called
  intercept_tcp_syn with a default forwarding option (connect to (ip,
  port))
- Endpoint.intercept_tcp_syn: also used by services like the HTTP proxy

The RST handling wasn't clear for the HTTP proxy etc, so rename
intercept_tcp_syn to intercept_tcp and ensure it handles RST packets
too.

Signed-off-by: David Scott <[email protected]>
We will have a future function which forwards to a Unix socket.

Signed-off-by: David Scott <[email protected]>
For now we only track connected TCP sockets, but in a future patch
we will add Unix sockets.

Prepare by tagging the socket with `Tcp socket and move the close
logic into the module.

Signed-off-by: David Scott <[email protected]>
This is modelled on the existing "Gateway_forwards"

Signed-off-by: David Scott <[email protected]>
This reads and watches the <json path> for configuration updates,
modelled on the existing `--gateway-forwards`

Signed-off-by: David Scott <[email protected]>
Define a simple handshake with a .json-formatted request and response.

Signed-off-by: David Scott <[email protected]>
This will allow testing of the I/O system.

Signed-off-by: David Scott <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant