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

[3.x] DSI tickle not being sent from server. #1514

Closed
NJRoadfan opened this issue Sep 22, 2024 · 4 comments · Fixed by #1517
Closed

[3.x] DSI tickle not being sent from server. #1514

NJRoadfan opened this issue Sep 22, 2024 · 4 comments · Fixed by #1517
Milestone

Comments

@NJRoadfan
Copy link
Contributor

This affects Netatalk 3.x and main branches. The server should be sending tickles over DSI every 30 seconds to a client so that the client knows the server is still up. Netatalk 2.x is not affected and sends out tickles every 30 seconds over DSI.

To reproduce:
-Install Netatalk 3.x or main branch.
-Monitor your network connection. You should see tickles from the client every minute or so, but none from the server.
-Connect to Netatalk using MacOS 8.x-9.x via TCP/IP. After 4 minutes, the client will time out and disconnect.

MacOS X clients don't seem to be adversely affected by this. They support client sleep/reconnect and can recover somewhat gracefully from the lack of server tickles. There is a function called alarm_handler() in afp_dsi.c that handles the periodic tickle transmissions. Somewhere along the line, it broke.

@rdmark
Copy link
Member

rdmark commented Sep 22, 2024

Good catch. I have observed this too intermittently, and now I understand the circumstances.

If it’s not too risky to fix, I think it’s worth patching stable 3.2 as well.

This was referenced Sep 22, 2024
@NJRoadfan
Copy link
Contributor Author

A meson configuration setting was wrong. Fixed in #1515 and #1517.

@rdmark rdmark added this to the release-3.2 milestone Sep 22, 2024
@rdmark
Copy link
Member

rdmark commented Sep 22, 2024

Autotools is not affected in 3.2?

@NJRoadfan
Copy link
Contributor Author

Doesn't appear so. This option should have always been 'no' or 'false'.

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 a pull request may close this issue.

2 participants