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

Too many Association requests issued over short period of time #155

Open
tznind opened this issue Jul 16, 2021 · 3 comments
Open

Too many Association requests issued over short period of time #155

tznind opened this issue Jul 16, 2021 · 3 comments

Comments

@tznind
Copy link
Contributor

tznind commented Jul 16, 2021

Describe the bug
The Hic device was attempting to associate with PACS and failing multiple times per second. This cumulative effect filled up the available Philips PACS Services and we were no longer able to accept any dicom requests

Expected behavior
System should log when associations are initiated or closed and detect when multiple associations are made over a short period of time. In such circumstance the system should shut down or enter a sleep mode until the problem is resolved.

@tznind tznind mentioned this issue Jul 16, 2021
@jas88
Copy link
Member

jas88 commented Jul 30, 2021

This issue might be relevant - seems to be a corner case where fo-dicom gets a TCP connection but doesn't get connected properly, so retries repeatedly in a loop? They reproduced it by trying to talk to Google as a PACS: it accepts the connection, but of course doesn't speak DICOM, so that stage fails - and gets retried repeatedly. fo-dicom/fo-dicom#933

@amoerie
Copy link

amoerie commented Feb 21, 2022

Hi, fo-dicom contributor here, I saw you guys linking to one of our issues. You might be interested in fo-dicom/fo-dicom#1285. Cheers!

@tznind
Copy link
Contributor Author

tznind commented Feb 21, 2022

Thanks for the heads up about the PR @amoerie. Being able to handle association request timeout as well as accepted/rejected should be helpful. At the very least we can add it into the count managed by our (rather brutal) MaximumAllowableAssociationEventsPerMinute setting in our tool.

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