# HG changeset patch # User Alain Mazy # Date 1776070574 -7200 # Node ID c3d9b76f6d0ae92f2835945f38caf70b76c59b0c # Parent b170a065c75f36271a82109bed0f48d1c5c7daa0 sync + 2.5.2 diff -r b170a065c75f -r c3d9b76f6d0a Aws/CMakeLists.txt --- a/Aws/CMakeLists.txt Mon Apr 13 10:55:59 2026 +0200 +++ b/Aws/CMakeLists.txt Mon Apr 13 10:56:14 2026 +0200 @@ -21,13 +21,13 @@ project(OrthancAwsS3Storage) -set(PLUGIN_VERSION "mainline") +set(PLUGIN_VERSION "2.5.2") if (PLUGIN_VERSION STREQUAL "mainline") set(ORTHANC_FRAMEWORK_DEFAULT_VERSION "mainline") set(ORTHANC_FRAMEWORK_DEFAULT_SOURCE "hg") else() - set(ORTHANC_FRAMEWORK_DEFAULT_VERSION "1.12.4") + set(ORTHANC_FRAMEWORK_DEFAULT_VERSION "1.12.10") set(ORTHANC_FRAMEWORK_DEFAULT_SOURCE "web") endif() diff -r b170a065c75f -r c3d9b76f6d0a Azure/CMakeLists.txt --- a/Azure/CMakeLists.txt Mon Apr 13 10:55:59 2026 +0200 +++ b/Azure/CMakeLists.txt Mon Apr 13 10:56:14 2026 +0200 @@ -21,13 +21,13 @@ project(OrthancAzureBlobStorage) -set(PLUGIN_VERSION "mainline") +set(PLUGIN_VERSION "2.5.2") if (PLUGIN_VERSION STREQUAL "mainline") set(ORTHANC_FRAMEWORK_DEFAULT_VERSION "mainline") set(ORTHANC_FRAMEWORK_DEFAULT_SOURCE "hg") else() - set(ORTHANC_FRAMEWORK_DEFAULT_VERSION "1.12.4") + set(ORTHANC_FRAMEWORK_DEFAULT_VERSION "1.12.10") set(ORTHANC_FRAMEWORK_DEFAULT_SOURCE "web") endif() diff -r b170a065c75f -r c3d9b76f6d0a Common/Resources/Orthanc/CMake/DownloadOrthancFramework.cmake --- a/Common/Resources/Orthanc/CMake/DownloadOrthancFramework.cmake Mon Apr 13 10:55:59 2026 +0200 +++ b/Common/Resources/Orthanc/CMake/DownloadOrthancFramework.cmake Mon Apr 13 10:56:14 2026 +0200 @@ -231,6 +231,11 @@ # for BlockingSharedMessageQueue.WaitEmpty() set(ORTHANC_FRAMEWORK_PRE_RELEASE ON) set(ORTHANC_FRAMEWORK_MD5 "c037cd2ddbe1b65b431692855483161b") + elseif (ORTHANC_FRAMEWORK_VERSION STREQUAL "56eb61c86f93") + # OE2 1.11.0 (framework post-1.12.10) + # for HttpClient that returns the answer body in case of HTTP error + set(ORTHANC_FRAMEWORK_PRE_RELEASE ON) + set(ORTHANC_FRAMEWORK_MD5 "665f8aa70d7c5091bc20da37cf664910") endif() endif() endif() diff -r b170a065c75f -r c3d9b76f6d0a Common/Resources/Orthanc/Plugins/OrthancPluginCppWrapper.cpp --- a/Common/Resources/Orthanc/Plugins/OrthancPluginCppWrapper.cpp Mon Apr 13 10:55:59 2026 +0200 +++ b/Common/Resources/Orthanc/Plugins/OrthancPluginCppWrapper.cpp Mon Apr 13 10:56:14 2026 +0200 @@ -1544,11 +1544,25 @@ #endif /* HAS_ORTHANC_PLUGIN_FIND_MATCHER == 1 */ + static void CheckAnswerSizeIsLessThan4GB(const std::string& answer) + { + if (answer.size() > static_cast(std::numeric_limits::max())) + { + #if HAS_ORTHANC_EXCEPTION == 1 + throw Orthanc::OrthancException(Orthanc::ErrorCode_ParameterOutOfRange, "Cannot send HTTP response larger than 4GB"); + #else + ORTHANC_PLUGINS_LOG_ERROR("Cannot send HTTP response larger than 4GB"); + ORTHANC_PLUGINS_THROW_PLUGIN_ERROR_CODE(OrthancPluginErrorCode_ParameterOutOfRange); + #endif + } + } + void AnswerJson(const Json::Value& value, OrthancPluginRestOutput* output) { std::string bodyString; - WriteStyledJson(bodyString, value); + WriteStyledJson(bodyString, value); + CheckAnswerSizeIsLessThan4GB(bodyString); OrthancPluginAnswerBuffer(GetGlobalContext(), output, bodyString.c_str(), bodyString.size(), "application/json"); } @@ -1556,6 +1570,7 @@ const char* mimeType, OrthancPluginRestOutput* output) { + CheckAnswerSizeIsLessThan4GB(answer); OrthancPluginAnswerBuffer(GetGlobalContext(), output, answer.c_str(), answer.size(), mimeType); } @@ -1564,6 +1579,26 @@ OrthancPluginSendHttpStatusCode(GetGlobalContext(), output, httpError); } + void AnswerHttpError(uint16_t httpError, + OrthancPluginRestOutput* output, + const std::string& answer, + const char* mimeType) + { + CheckAnswerSizeIsLessThan4GB(answer); + + OrthancPluginSetHttpHeader(GetGlobalContext(), + output, + "content-type", + mimeType); + + OrthancPluginSendHttpStatus(GetGlobalContext(), + output, + httpError, + answer.c_str(), + static_cast(answer.size())); + } + + void AnswerMethodNotAllowed(OrthancPluginRestOutput *output, const char* allowedMethods) { OrthancPluginSendMethodNotAllowed(GetGlobalContext(), output, allowedMethods); diff -r b170a065c75f -r c3d9b76f6d0a Common/Resources/Orthanc/Plugins/OrthancPluginCppWrapper.h --- a/Common/Resources/Orthanc/Plugins/OrthancPluginCppWrapper.h Mon Apr 13 10:55:59 2026 +0200 +++ b/Common/Resources/Orthanc/Plugins/OrthancPluginCppWrapper.h Mon Apr 13 10:56:14 2026 +0200 @@ -690,6 +690,11 @@ void AnswerHttpError(uint16_t httpError, OrthancPluginRestOutput* output); + void AnswerHttpError(uint16_t httpError, + OrthancPluginRestOutput* output, + const std::string& answer, + const char* mimeType); + void AnswerMethodNotAllowed(OrthancPluginRestOutput* output, const char* allowedMethods); #if ORTHANC_PLUGINS_VERSION_IS_ABOVE(1, 5, 0) @@ -1025,7 +1030,20 @@ OrthancPluginSetMetricsValue(GetGlobalContext(), name, value, OrthancPluginMetricsType_Default); } +#endif + +#if ORTHANC_PLUGINS_VERSION_IS_ABOVE(1, 12, 1) + inline void SetMetricsValue(const char* name, + int64_t value) + { + OrthancPluginSetMetricsIntegerValue(GetGlobalContext(), name, + value, OrthancPluginMetricsType_Default); + } +#endif + + +#if HAS_ORTHANC_PLUGIN_METRICS == 1 class MetricsTimer : public boost::noncopyable { private: diff -r b170a065c75f -r c3d9b76f6d0a Google/CMakeLists.txt --- a/Google/CMakeLists.txt Mon Apr 13 10:55:59 2026 +0200 +++ b/Google/CMakeLists.txt Mon Apr 13 10:56:14 2026 +0200 @@ -21,13 +21,13 @@ project(OrthancGoogleCloudStorage) -set(PLUGIN_VERSION "mainline") +set(PLUGIN_VERSION "2.5.2") if (PLUGIN_VERSION STREQUAL "mainline") set(ORTHANC_FRAMEWORK_DEFAULT_VERSION "mainline") set(ORTHANC_FRAMEWORK_DEFAULT_SOURCE "hg") else() - set(ORTHANC_FRAMEWORK_DEFAULT_VERSION "1.12.4") + set(ORTHANC_FRAMEWORK_DEFAULT_VERSION "1.12.10") set(ORTHANC_FRAMEWORK_DEFAULT_SOURCE "web") endif() diff -r b170a065c75f -r c3d9b76f6d0a NEWS --- a/NEWS Mon Apr 13 10:55:59 2026 +0200 +++ b/NEWS Mon Apr 13 10:56:14 2026 +0200 @@ -1,5 +1,5 @@ -Pending changes in the mainline -=============================== +2026-04-13 - v 2.5.2 +==================== * AWS plugin: * "ConnectTimeout" has been renamed into "ConnectionTimeout" to match the documentation.