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

feat: direct connection callback #8

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Conversation

ramfox
Copy link
Contributor

@ramfox ramfox commented Apr 10, 2024

magic_endpoint_conn_type_cb calls a callback for each ConnectionType event we receive.

main.c uses magic_endpoint_conn_type_cb on the client side.

@ramfox ramfox marked this pull request as ready for review April 10, 2024 01:28
@ramfox ramfox self-assigned this Apr 10, 2024
@ramfox ramfox requested a review from dignifiedquire April 10, 2024 01:28
ramfox added 3 commits April 15, 2024 20:48
first attempt for some kind of callback API for when a direct connection is established between an endpoint and a peer
@ramfox ramfox force-pushed the ramfox/direct_conn_cb branch from a206749 to dc3e308 Compare April 17, 2024 04:14
@ramfox ramfox requested a review from dignifiedquire April 22, 2024 14:16
/// Does not block. The provided callback will be called when we have a direct
/// connection to the peer associated with the `node_id`, or the timeout has occurred.
///
/// To wait indefinitely, provide -1 for the timeout parameter.
Copy link
Contributor

Choose a reason for hiding this comment

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

what timeout parameter?

@dignifiedquire
Copy link
Contributor

want to rebase, so we can get this in?

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