Skip to content
This repository has been archived by the owner on May 26, 2022. It is now read-only.

Implement CloseRead/CloseWrite #174

Merged
merged 1 commit into from
Sep 3, 2020
Merged

Implement CloseRead/CloseWrite #174

merged 1 commit into from
Sep 3, 2020

Conversation

Stebalien
Copy link
Member

@Stebalien Stebalien commented Aug 28, 2020

This:

  • Changes Close to behave like the unix close (sends an EOF, ignores future inbound data).
  • Adds a CloseWrite to replace the current Close.
  • Adds a CloseRead to close the read side, while leaving the write side open.

See libp2p/go-libp2p-core#166.
Depends on libp2p/go-libp2p-core#166, don't merge until that's merged.

Copy link
Collaborator

@marten-seemann marten-seemann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but this will break #173.

@Stebalien
Copy link
Member Author

We'll probably need to write interface shims anyways.

This:

* Changes `Close` to behave like the unix `close` (sends an EOF, ignores future
inbound data).
* Adds a `CloseWrite` to replace the current `Close`.
* Adds a `CloseRead` to close the read side, while leaving the write side open.

See libp2p/go-libp2p-core#166.
@Stebalien
Copy link
Member Author

@marten-seemann could you merge this when you're ready?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants