Skip to content

fix(daemon): log repeated admission timeouts at debug, first at info - #296

Closed
kiwipaulrob wants to merge 0 commit into
Sendspin:mainfrom
kiwipaulrob:fix/admission-timeout-log-once
Closed

kiwipaulrob wants to merge 0 commit into
Sendspin:mainfrom
kiwipaulrob:fix/admission-timeout-log-once

Conversation

@kiwipaulrob

Copy link
Copy Markdown
Contributor

What does this implement/fix?

When a server keeps redialing, every unadmitted connection logs the same info line, so the daemon log fills with repeats of one event:

INFO:sendspin.daemon.daemon:Incoming connection did not complete admission (handshake timed out or was aborted); waiting for a retry.

I hit this watching my own player logs, where the line repeats on every reconnect and buries everything else. This keeps the first occurrence at info (an unadmitted connection would otherwise be fully silent, as the code comment notes) and moves repeats to debug. The flag resets once a connection is admitted, so a later wedged spell is visible again at info.

No behavior change apart from log level: three lines of state (one bool, set on first log, cleared on admission).

Related issue (if applicable):

  • none filed — happy to open one if you would rather track it that way.

Types of changes

  • Bugfix (non-breaking change which fixes an issue): bugfix
  • New feature (non-breaking change which adds functionality): new-feature
  • Enhancement to an existing feature: enhancement
  • Breaking change (changes CLI options, settings, hooks or the protocol in a way that is not backwards compatible): breaking-change
  • Refactor (no behavior change): refactor
  • Documentation only: documentation
  • Maintenance / chore: maintenance
  • CI / workflow change: ci
  • Dependencies bump: dependencies

@github-actions github-actions Bot added the bugfix Fixes a bug label Oct 17, 2026
@kiwipaulrob
kiwipaulrob force-pushed the fix/admission-timeout-log-once branch from 41f98cf to 0d8c335 Compare October 17, 2026 03:39

This branch was successfully deployed

1 active deployment
publish — 0d8c3353 Deployed Oct 12, 2026 by maximmaxim345 via Upload release to PyPI #54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Fixes a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant