Skip to content

feat: Add connection timeout options via CLI flags to DIMSE Tools#792

Open
karanrane96 wants to merge 1 commit into
Enet4:masterfrom
karanrane96:feat/connection-timeout-options
Open

feat: Add connection timeout options via CLI flags to DIMSE Tools#792
karanrane96 wants to merge 1 commit into
Enet4:masterfrom
karanrane96:feat/connection-timeout-options

Conversation

@karanrane96

Copy link
Copy Markdown
  • [app-common] add ConnectionOptions struct with --read-timeout and --write-timeout flags, shared across
    all tools
  • [echoscu] add dicom-app-common dependency; add --connect-timeout, --read-timeout, --write-timeout flags
  • [findscu] add dicom-app-common dependency; add --connect-timeout, --read-timeout, --write-timeout flags
  • [storescu] add --connect-timeout, --read-timeout, --write-timeout flags; wire timeouts in both sync and
    async paths
  • [storescp] add --read-timeout, --write-timeout flags (no --connect-timeout — server does not initiate
    TCP connections)

Notes:

  • --connect-timeout sets a deadline on TCP connection establishment (TcpStream::connect_timeout),
    preventing indefinite hangs when the remote host is unreachable
  • --read-timeout / --write-timeout bound socket I/O, preventing hangs on stalled associations
  • All timeout values are in whole seconds

@Enet4 Enet4 added enhancement A-tool Area: tooling C-storescu Crate: dicom-storescu C-storescp Crate: dicom-storescp C-findscu Crate: dicom-findscu C-echoscu Crate: dicom-echoscu A-app-common Library crate dicom-app-common labels Jun 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-app-common Library crate dicom-app-common A-tool Area: tooling C-echoscu Crate: dicom-echoscu C-findscu Crate: dicom-findscu C-storescp Crate: dicom-storescp C-storescu Crate: dicom-storescu enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants