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

Restore local peer discovery mechanism on Android 11+ #1158

Merged
merged 1 commit into from
Aug 16, 2024

Conversation

bobrofon
Copy link
Contributor

@bobrofon bobrofon commented Aug 6, 2024

This solution is bases on https://github.com/wlynxg/anet project. github.com/wlynxg/anet is a partial alternative implementation of the golang.org/x/net module. The goal of anet module is to provide workarounds of the issues golang/go#40569 and golang/go#68082 on Android 11+.

Tested on AOSP 13.

Resolves: #1149

@Revertron
Copy link
Contributor

@neilalexander are you good with this dependency?

'github.com/wlynxg/anet' is a partial alternative implementation of
the 'golang.org/x/net' module. The goal of 'anet' module is to provide
workarounds of the issues golang/go#40569 and golang/go#68082 on
Android 11+.
@bobrofon
Copy link
Contributor Author

bobrofon commented Aug 13, 2024

It looks like bobrofon@cfb329b is no longer necessary after the link and multicast fixes were merged into version 0.5.8. At least, I cannot reproduce the livelock on multicasting stop anymore. Removed this commit from the PR.

Copy link
Member

@neilalexander neilalexander left a comment

Choose a reason for hiding this comment

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

I checked over the anet library and it looks like a nice thin wrapper, so I’m OK with this if you are happy it solves the Android problems.

@neilalexander neilalexander merged commit 947b6ad into yggdrasil-network:develop Aug 16, 2024
15 checks passed
@bobrofon bobrofon deleted the android-multicast branch August 22, 2024 20:17
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.

Change multicast code for Android
3 participants