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

problem when renewing connexion #32

Open
jeremyczaicki opened this issue Mar 26, 2020 · 15 comments
Open

problem when renewing connexion #32

jeremyczaicki opened this issue Mar 26, 2020 · 15 comments

Comments

@jeremyczaicki
Copy link

Hi,
I use qzeroconf in my app to discover Cisco devices. when i launch my app and start browser, i can get all my devices connected to the network, but when i unplug a device:
1)removeService is emitted
2) if i plug another device, i cannot see it
3)if i replug the device i unplugged, service does not appear anymore.

to resume, when i just unplug a device, no more service is added.

i am developing on Mac OS Mojave

Do you have a solution ?

@jeremyczaicki
Copy link
Author

i have to stop and start browser 3 times to find my devices again, but even if one device is unplugged, it found it. can you help me ?

@jbagg
Copy link
Owner

jbagg commented Mar 28, 2020

Hi Jeremy,

I do not have any mac hardware at the moment as I am working from home due to covid-19. Could you please run the example and see if it has the same behaviour. Please also remember that TTL records are typically 120 seconds

@jeremyczaicki
Copy link
Author

Hi,
i ran exemple and it is the same issue. Yes, i see that TTL is 120 sec on wireshark. But if i stop browser and i start it again ? is there any solution to empty the cache ? the problem is also that if a device is removed i cannot add it anymore

@jeremyczaicki
Copy link
Author

here, is my function to refresh mDNS browser:

void MainWindow::on_buttonn_refresh_clicked(){
if(zeroConf.browserExists()){
zeroConf.stopBrowser();
zeroConf.startBrowser(tr("_csco-sb._tcp"));
}
else if(!zeroConf.browserExists()){
zeroConf.startBrowser(tr("_csco-sb._tcp"));
}
}

is it correct ? because i have to click on 3 times before i can see my devices

@mincequi
Copy link

Same here:
Apparently the resolverCallback is not called when calling: DNSServiceResolve (bonjour.cpp:88).

When I trigger startBrowser() 3 times, it works again.

@jbagg
Copy link
Owner

jbagg commented Mar 29, 2020 via email

@mincequi
Copy link

mincequi commented Mar 30, 2020

8625824 "Bonjour - need to keep resolvers around for updates"

NOK

c286fcd "Change browser protocol options so it can only be IPv4 or IPv6, not both."

OK

Feel free to send patches for testing.

@jeremyczaicki
Copy link
Author

sorry but i am not sure to understand what you want me to do ?

@jbagg
Copy link
Owner

jbagg commented Mar 30, 2020

Manuel (mincequi) found this issue is a result of commit 8625824. I don't need you to do anything at the moment. I am going into my work tomorrow to pick up some things. I can grab the old mac I've been testing with. You can use c286fcd for the time being. By rolling back to this older commit, you will loose the functionality of IP address updates if the IP address of a device changes.

@jeremyczaicki
Copy link
Author

into avahiclient.cpp, there is #include librairies that are not founded, such as : avahi-client/client.h, avahi-client/publish.h avahi-client/lookup.h. actually, there is no file named "avahi-client" in the downloaded file.

@jbagg
Copy link
Owner

jbagg commented Mar 30, 2020 via email

@jeremyczaicki
Copy link
Author

Hi jbagg,
have you found a solution ?

@jbagg
Copy link
Owner

jbagg commented Apr 11, 2020

Yes, please type
git checkout c286fcd

I tried to re-install El capitan on my companies macbook, but when I enter my apple id, it fails. (and I already erased the drive) I can login to the web, but not the installer. Good old apple quality! I've thought about buying a newer mac, but I think I would be throwing away my time and money. Apple just isn't friendly to developers (or anyone for that matter)

@mincequi
Copy link

Is there an Update on this issue? As mentioned before, i can test that for you.

@jbagg
Copy link
Owner

jbagg commented Oct 30, 2020

yes, this should be fixed in commit 9d7b114 (July 8th, 2020). Please test.

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

No branches or pull requests

3 participants