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

Milestone-1 TCP-Transport. #536

Closed

Conversation

ayuryshev
Copy link
Collaborator

Excluding the last task defined in skycoin/skywire#501 - everything is completed.
It was already done by most part a long time before.
The most hard part is in the last task.

It's a part of bigger and unfinished tcp-transport PR.

Here:
- Fix of tcp-transport itself - it did not send laddr during Dial so
on Accept on another edge was broken
- Fix of memoryPubKeyTable: reverse search of pk was broken
- FilePubKeyTable - simplified, now composed from memoryPubKeyTable
- tests of tcp-transport - under no_ci tags because they need prepared IP addresses
- .travis-ci now prepares IP aliases for tests
Excluding the last task defined  in skycoin/skywire#501 - everything is completed.

It was already done by most part a long time before.

The most hard part is in the last task.
pkg/snet/network.go Outdated Show resolved Hide resolved
pkg/snet/network.go Outdated Show resolved Hide resolved
pkg/snet/network.go Outdated Show resolved Hide resolved
pkg/snet/network.go Outdated Show resolved Hide resolved
pkg/snet/tcp_transport.go Outdated Show resolved Hide resolved
pkg/snet/tcp_transport.go Outdated Show resolved Hide resolved
pkg/snet/tcp_transport.go Outdated Show resolved Hide resolved
pkg/snet/tcp_transport.go Outdated Show resolved Hide resolved
pkg/snet/tcp_transport.go Outdated Show resolved Hide resolved
pkg/snet/tcp_transport.go Outdated Show resolved Hide resolved
@ayuryshev
Copy link
Collaborator Author

Fixes requested by Evan - done.
Linter errors and failed tests that were before this PR remains the same.

@ayuryshev
Copy link
Collaborator Author

Remerged with mainnet-milestone-1. Fixed tests for tcp-transport

@evanlinjin evanlinjin mentioned this pull request Sep 4, 2019
@evanlinjin evanlinjin closed this Sep 5, 2019
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