Skip to content

cjdns-admin use unix socket to communicate with cjdns#36

Open
cjdelisle wants to merge 1 commit intomasterfrom
admin-unix-socket
Open

cjdns-admin use unix socket to communicate with cjdns#36
cjdelisle wants to merge 1 commit intomasterfrom
admin-unix-socket

Conversation

@cjdelisle
Copy link
Copy Markdown
Collaborator

fallback on UDP if unix socket not working

fallback on UDP if unix socket not working
@a-kordys a-kordys self-requested a review November 19, 2020 14:20
@a-kordys
Copy link
Copy Markdown
Collaborator

Looks good overall. A few notes.

  • Compatibility with Windows is totally broken now. Need to put a couple of #[cfg(unix)] here and there to conditionally enable unix sockets under Unix only.
  • Suspiciously too many changes in Cargo.lock - probably some old version of rust is used?

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.

2 participants