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

[findscu] Add response status check in C-FIND-RSP response data set #434

Merged
merged 2 commits into from
Nov 9, 2023

Conversation

Enet4
Copy link
Owner

@Enet4 Enet4 commented Nov 3, 2023

Fixes a situation where the SCU would wait eternally for more messages if the SCP sent a command set with the status Pending but then responded with status 0 in the following data set.

@Enet4 Enet4 added bug This is a bug A-tool Area: tooling C-findscu Crate: dicom-findscu labels Nov 3, 2023
@Enet4 Enet4 changed the title [findscu] Add response status check in C-FIND-RSP [findscu] Add response status check in C-FIND-RSP response data set Nov 3, 2023
fixes a situation where the SCU would
wait eternally for more messages
if the SCP sent a command set with pending status
but then responded with 0 in the data set
@Enet4 Enet4 force-pushed the imp/findscu/check-status-data branch from 7e62f4d to 8dd6fd4 Compare November 5, 2023 18:29
findscu/src/main.rs Outdated Show resolved Hide resolved
@Enet4 Enet4 merged commit 213ccab into master Nov 9, 2023
4 checks passed
@Enet4 Enet4 deleted the imp/findscu/check-status-data branch November 9, 2023 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-tool Area: tooling bug This is a bug C-findscu Crate: dicom-findscu
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant