# HG changeset patch
# User Sebastien Jodogne <s.jodogne@gmail.com>
# Date 1554462177 -7200
# Node ID 8f44dadcdbc1a86446b87a8bcdfbceb0af008174
# Parent  ca65e3dcce724e14325c0089982849af5fe63bcd
Fix issue #138 (POST to modalities/{name} accepts invalid characters)

diff -r ca65e3dcce72 -r 8f44dadcdbc1 NEWS
--- a/NEWS	Thu Apr 04 16:03:29 2019 +0200
+++ b/NEWS	Fri Apr 05 13:02:57 2019 +0200
@@ -2,6 +2,7 @@
 ===============================
 
 * Size of the Orthanc static binaries are reduced by compressing ICU data
+* Fix issue #138 (POST to modalities/{name} accepts invalid characters)
 
 
 Version 1.5.6 (2019-03-01)
diff -r ca65e3dcce72 -r 8f44dadcdbc1 OrthancServer/OrthancConfiguration.cpp
--- a/OrthancServer/OrthancConfiguration.cpp	Thu Apr 04 16:03:29 2019 +0200
+++ b/OrthancServer/OrthancConfiguration.cpp	Fri Apr 05 13:02:57 2019 +0200
@@ -759,6 +759,8 @@
   void OrthancConfiguration::UpdateModality(const std::string& symbolicName,
                                             const RemoteModalityParameters& modality)
   {
+    CheckAlphanumeric(symbolicName);
+    
     modalities_[symbolicName] = modality;
     SaveModalities();
   }
@@ -774,6 +776,8 @@
   void OrthancConfiguration::UpdatePeer(const std::string& symbolicName,
                                         const WebServiceParameters& peer)
   {
+    CheckAlphanumeric(symbolicName);
+    
     peer.CheckClientCertificate();
 
     peers_[symbolicName] = peer;