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

Draft: Wire Protocol Overhaul, Message Handling Refactor #2204

Merged
merged 32 commits into from Oct 16, 2023

Conversation

dr7ana
Copy link
Collaborator

@dr7ana dr7ana commented Aug 29, 2023

No description provided.

llarp/util/buffer.hpp Show resolved Hide resolved
llarp/router_contact.cpp Outdated Show resolved Hide resolved
llarp/router/i_outbound_session_maker.hpp Outdated Show resolved Hide resolved
llarp/router/abstractrouter.hpp Outdated Show resolved Hide resolved
llarp/router/abstractrouter.hpp Outdated Show resolved Hide resolved
llarp/messages/link_message.hpp Outdated Show resolved Hide resolved
llarp/messages/link_message.hpp Outdated Show resolved Hide resolved
@dr7ana dr7ana force-pushed the quic-wip branch 3 times, most recently from 45c2be3 to 1dc877e Compare August 30, 2023 19:15
llarp/dht/messages/pubintro.cpp Outdated Show resolved Hide resolved
llarp/messages/common.hpp Show resolved Hide resolved
- routing messages and surrounding code
- shim code in place for iteration and optimization after deciding what to do with buffer, string handling, and subsequent function calls
- `::handle_message` is transposed; Rather than the message calling the method and taking a reference to the router, the router should have a handle_message method and take a reference to the message
- `::EndcodeBuffer` takes a string reference, to which the result of `::bt_encode()` is assigned
- added CI fixes for gnutls build
- stupid warn deprecated stupid
RIP:
- i_link_manager
- i_outbound_message_handler
- i_gossiper
- i_outbound_session_maker
- i_rc_lookup_handler
@dr7ana dr7ana mentioned this pull request Sep 14, 2023
llarp/link/endpoint.cpp Outdated Show resolved Hide resolved
- bumped libquic to begin implementing dependent features on connection open/close signals
- more gutting of interface classes
- llarp/router/router.hpp, route_poker, and platform code moved to libquic Address types
- implementing required methods in link_manager for connection establishment
- coming along nicely
@dr7ana dr7ana force-pushed the quic-wip branch 2 times, most recently from 9d172ec to 84bdee6 Compare October 2, 2023 20:06
- tons of surrounding stupid refactoring required
dr7ana and others added 3 commits October 6, 2023 08:35
- free standing functions in link_manager
- sign and serialize functionalities siloed in llarp/messages/exit.hpp
- implementing DHT message transmission methods and surrounding functionalities across paths
llarp/link/link_manager.cpp Outdated Show resolved Hide resolved
llarp/link/link_manager.cpp Outdated Show resolved Hide resolved
llarp/link/link_manager.cpp Outdated Show resolved Hide resolved
llarp/link/link_manager.cpp Outdated Show resolved Hide resolved
llarp/link/link_manager.cpp Outdated Show resolved Hide resolved
llarp/link/link_manager.cpp Outdated Show resolved Hide resolved
llarp/link/link_manager.cpp Outdated Show resolved Hide resolved
- Reworking how paths to services and snodes
- pushing for Tom to rebase for path handling
- Added call_get to ev.hpp to queue event loop operations w/ a return value
- de-mutexed NodeDB and made all operations via event loop. Some calls to NodeDB methods (like ::put_if_newer) were wrapped in call->get's, but some weren't. All function bodies were using mutex locks
- mutexes in nodedb and rc_lookup_handler removed
- implemented thread-safe queueing to event loop for previously locked accesses
@dr7ana dr7ana marked this pull request as ready for review October 16, 2023 18:05
@jagerman
Copy link
Member

Merging into quic-wip not because it's done, but because that's already a WIP branch, and ought to make new additions/updates/reviews easier of more code going into quic-wip.

@jagerman jagerman merged commit 690ec5f into oxen-io:quic-wip Oct 16, 2023
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.

None yet

4 participants