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

Use lightning-net-tokio for non-wasm builds #1077

Merged
merged 1 commit into from
Mar 25, 2024
Merged

Conversation

benthecarman
Copy link
Collaborator

@benthecarman benthecarman commented Mar 23, 2024

was helping @elnosh with getting networking working for mutiny-server

tried a bunch of things and gave up, just using lightning-net-tokio instead of rolling our own network impl for server context

@benthecarman benthecarman added bug Something isn't working proxy/p2p labels Mar 23, 2024
@benthecarman
Copy link
Collaborator Author

Don't merge yet, waiting for @elnosh to say everything is working for him with this

@elnosh
Copy link
Contributor

elnosh commented Mar 24, 2024

Thanks for the help here. It fixed some of the problems I was having but still running into some intermittent issues while starting the server so I wouldn't merge this yet. I'll keep testing this

@benthecarman benthecarman force-pushed the fix-std-net branch 3 times, most recently from c2173db to f1afede Compare March 24, 2024 20:13
@benthecarman benthecarman changed the title Fixes for non-wasm networking Use lightning-net-tokio for non-wasm builds Mar 24, 2024
@benthecarman
Copy link
Collaborator Author

Was still getting issues so just switched to using lightning-net-tokio when in non-wasm

@elnosh
Copy link
Contributor

elnosh commented Mar 24, 2024

I've been trying now with your changes using lightning-net-tokio and so far no issues for me.

Copy link
Contributor

@TonyGiorgio TonyGiorgio left a comment

Choose a reason for hiding this comment

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

Seems contained to just the server code, so ack from me if it works.

@benthecarman benthecarman merged commit eee52a7 into master Mar 25, 2024
9 checks passed
@benthecarman benthecarman deleted the fix-std-net branch March 25, 2024 18:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working proxy/p2p
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants