Mercurial > hg > orthanc

diff OrthancServer/Internals/CommandDispatcher.cpp @ 2352:3ab96768d144

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Fix issue #52 (DICOM level security association problems)
author Sebastien Jodogne <s.jodogne@gmail.com>
date Tue, 18 Jul 2017 17:33:26 +0200
parents a3a65de1840f
children b8969010b534
line wrap: on
line diff
--- a/OrthancServer/Internals/CommandDispatcher.cpp	Tue Jul 18 08:41:09 2017 +0200
+++ b/OrthancServer/Internals/CommandDispatcher.cpp	Tue Jul 18 17:33:26 2017 +0200
@@ -806,13 +806,12 @@
 
         // Check whether this request is allowed by the security filter
         if (supported && 
-            request != DicomRequestType_Echo &&  // Always allow incoming ECHO requests
             filter_ != NULL &&
             !filter_->IsAllowedRequest(remoteIp_, remoteAet_, calledAet_, request))
         {
-          LOG(ERROR) << EnumerationToString(request) 
-                     << " requests are disallowed for the AET \"" 
-                     << remoteAet_ << "\"";
+          LOG(WARNING) << "Rejected " << EnumerationToString(request)
+                       << " request from remote DICOM modality with AET \""
+                       << remoteAet_ << "\" and hostname \"" << remoteIp_ << "\"";
           cond = DIMSE_ILLEGALASSOCIATION;
           supported = false;
           finished = true;
@@ -906,7 +905,7 @@
         else
         {
           OFString temp_str;
-          LOG(ERROR) << "DIMSE failure (aborting association): " << cond.text();
+          LOG(INFO) << "DIMSE failure (aborting association): " << cond.text();
           /* some kind of error so abort the association */
           ASC_abortAssociation(assoc_);
         }