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

mDNS search does not work on ios #29

Open
guyluz11 opened this issue Dec 2, 2023 · 5 comments
Open

mDNS search does not work on ios #29

guyluz11 opened this issue Dec 2, 2023 · 5 comments
Assignees

Comments

@guyluz11
Copy link
Contributor

guyluz11 commented Dec 2, 2023

As written it this issue multicast_dns currently has an error for mDNS search on ios (didn't test myself).

I saw other developers use bonsoir for Android and iOS.
Maybe we can integrate it like that as well, android/ iOS to use Bonsoir and others to use multicast_dns until the issue is resolved.

@git-elliot
Copy link
Contributor

We can not keep on changing implementations just because flutter is a crap framework which keeps on breaking..

@guyluz11
Copy link
Contributor Author

guyluz11 commented Dec 4, 2023

Ha ha ok, so we will keep multicast_dns and it will solve itself in time

@guyluz11 guyluz11 closed this as completed Dec 4, 2023
@guyluz11
Copy link
Contributor Author

guyluz11 commented Dec 30, 2023

It seems that that time come for me to require this functionality and I don't have a problem implementing it myself.
Solving network issues for specific platforms was one of the reasons that we created the network_tools_flutter package.

bonsoir can be our solution to mDNS on Flutter in general for now not just Android and IOS, so one implementation here.
Will follow the same structure that you did before, the same class and methods but with the word flutter at the end.

I will open a pr and if you are not satisfied with it it is ok and I will just implement it only in my project.
But I think in general it solves a requirement for network requests so I think this is the place for it.

@guyluz11 guyluz11 reopened this Dec 30, 2023
@git-elliot
Copy link
Contributor

If it's not resolved till now then we have to do something... It's all about which project is maintained highly. Let's solve it ourselves. Please go ahead and let me know if anything is required.

@guyluz11
Copy link
Contributor Author

Pr tomorrow 😁

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: In Progress
Development

Successfully merging a pull request may close this issue.

2 participants