-
Notifications
You must be signed in to change notification settings - Fork 9
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
Stratum v2 connman #50
base: 2024/06/sv2_transport
Are you sure you want to change the base?
Conversation
@pinheadmz wrote in the original PR:
I plan to take a look at this. |
3cf779b
to
b549fd6
Compare
bdc8542
to
199234a
Compare
b549fd6
to
22d033c
Compare
199234a
to
922fd8a
Compare
22d033c
to
6b5ee20
Compare
CMake rebase. |
6b5ee20
to
4f957ee
Compare
922fd8a
to
50b6eb8
Compare
4f957ee
to
b69544c
Compare
50b6eb8
to
6a02367
Compare
b69544c
to
1ca68d2
Compare
6a02367
to
ce4269c
Compare
1ca68d2
to
b578e0a
Compare
8220337
to
5fca2c5
Compare
b578e0a
to
489c9fb
Compare
5fca2c5
to
dfb9a6d
Compare
489c9fb
to
a55fa83
Compare
Rebased to move everything into a |
dfb9a6d
to
4c680d6
Compare
a55fa83
to
5c28af1
Compare
4c680d6
to
e28078f
Compare
Co-Authored-By: Christopher Coverdale <chris.coverdale24@gmail.com>
Move the implementation (method definitions) from `test/util/net.h` to `test/util/net.cpp` to make the header easier to follow.
…lass This allows reusing them in other mocked implementations.
…o it And also allows gradually providing the data to be returned by `Recv()` and sending and receiving net messages (`CNetMessage`).
Co-Authored-By: Vasil Dimov <vd@FreeBSD.org>
Similar to e9bfbb5 for Cirrus, but not opt-in, because Github CI lacks custom variables.
5c28af1
to
53f852b
Compare
Moved to fork repo from bitcoin#30332 (comment)
Based on bitcoin#30315 and bitcoin#30205. Followed by #49.
Parent PR bitcoin#29432.
This PR introduces
Sv2Connman
which is somewhat similar toCConnman
. It uses theSv2Transport
introduced in bitcoin#30315 to enable incoming connections from other Stratum v2 roles.It's main target user is the Template Provider role introduced in #49.
There may be other Stratum v2 roles we want to support in the future.
A remaining issue is that the code in
ThreadSv2Handler
reuses a lot of code fromCConnman::SocketHandlerConnected
. I'd like to find a way to extract this common functionality and put it somewhere else.TODO:
CConnman::SocketHandlerConnected
(partially) reusablebitcoin_node
(e.g. move Transport tobitcoin_common
)