Fix issue with escape_json

Author: Deniskore

binlog_json_parser/CMakeLists.txt

@@ -5,9 +5,19 @@ project(binlog_json_parser)
 set(CMAKE_CXX_STANDARD 23)
 set(CMAKE_CXX_STANDARD_REQUIRED ON)
 
+include(CheckIPOSupported)
+include(CheckCSourceCompiles)
+check_ipo_supported(RESULT IPO_SUPPORTED OUTPUT IPO_OUTPUT)
+
+if(IPO_SUPPORTED)
+    set(CMAKE_INTERPROCEDURAL_OPTIMIZATION TRUE)
+    message(STATUS "Interprocedural optimization (IPO/LTO) enabled globally.")
+else()
+    message(STATUS "IPO/LTO is not supported: ${IPO_OUTPUT}")
+endif()
+
 # Check if the build type is Release
 if(CMAKE_BUILD_TYPE STREQUAL "Release")
-
     # Set optimization level to -O3 for release builds
     if(NOT CMAKE_CXX_FLAGS_RELEASE MATCHES "-O")
         set(CMAKE_CXX_FLAGS_RELEASE "${CMAKE_CXX_FLAGS_RELEASE} -O3")
@@ -32,24 +42,35 @@ if(CMAKE_BUILD_TYPE STREQUAL "Release")
         message(WARNING "The -march option will not be set because the system is not x86 or x64.")
     endif()
 
-    # Check for LTO support
-    include(CheckCXXCompilerFlag)
-
-    check_cxx_compiler_flag("-flto" COMPILER_SUPPORTS_LTO)
-
-    if(COMPILER_SUPPORTS_LTO)
-        message(STATUS "Link Time Optimization (LTO) is supported by the compiler.")
-        set(CMAKE_CXX_FLAGS_RELEASE "${CMAKE_CXX_FLAGS_RELEASE} -flto")
-        set(CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS} -flto")
-        set(CMAKE_SHARED_LINKER_FLAGS "${CMAKE_SHARED_LINKER_FLAGS} -flto")
-    else()
-        message(WARNING "Link Time Optimization (LTO) is not supported by the compiler.")
-    endif()
-
     # Export compile flags to a file
     file(WRITE "${CMAKE_BINARY_DIR}/compile_flags.txt" "CXXFLAGS: ${CMAKE_CXX_FLAGS_RELEASE}\n")
     file(APPEND "${CMAKE_BINARY_DIR}/compile_flags.txt" "LINKER_FLAGS: ${CMAKE_EXE_LINKER_FLAGS}\n")
-
 endif()
 
-add_library(mysqljsonparse SHARED mysqljsonparse.cpp mysql_json_parser.cpp)
+check_c_source_compiles("
+#include <features.h>
+#if defined(__GLIBC__)
+#error \"This is glibc, not musl\"
+#endif
+#include <stdio.h>
+int main() { return 0; }
+" IS_MUSL)
+
+option(ALPINE_STATIC "Force fully static build when using musl/Alpine" ${IS_MUSL})
+
+if(ALPINE_STATIC)
+    add_definitions(-D_FORTIFY_SOURCE=0)
+    message(STATUS "musl detected → producing shared library with static musl linking")
+    
+    add_library(mysqljsonparse SHARED mysqljsonparse.cpp mysql_json_parser.cpp)
+    target_link_options(mysqljsonparse PRIVATE 
+        -static
+        -static-libgcc 
+        -static-libstdc++
+        -fPIC
+    )
+    target_compile_options(mysqljsonparse PRIVATE -fPIC)
+else()
+    message(STATUS "musl not detected → building shared library with dynamic glibc")
+    add_library(mysqljsonparse SHARED mysqljsonparse.cpp mysql_json_parser.cpp)
+endif()

binlog_json_parser/Dockerfile

@@ -0,0 +1,25 @@
+FROM alpine:3.22 AS build
+
+RUN apk add --no-cache \
+        build-base \
+        clang \
+        lld \
+        cmake \
+        ninja \
+        musl-dev
+
+WORKDIR /src
+
+COPY . .
+
+RUN cmake -S . -B build -G Ninja \
+        -DCMAKE_BUILD_TYPE=Release \
+        -DCMAKE_C_COMPILER=clang \
+        -DCMAKE_CXX_COMPILER=clang++ \
+        && ninja -C build -v
+
+FROM scratch AS artifact
+
+COPY --from=build /src/build/libmysqljsonparse.so /
+
+CMD [""]

binlog_json_parser/build_static.sh

@@ -0,0 +1,21 @@
+#!/usr/bin/env bash
+set -euo pipefail
+
+IMAGE_TAG=${1:-mysqljsonparse:alpine-static}
+ARTIFACT=${2:-libmysqljsonparse.so}
+
+echo "[INFO] Building Docker image '${IMAGE_TAG}'..."
+docker build -t "${IMAGE_TAG}" .
+
+echo "[INFO] Creating temporary container from image..."
+CID=$(docker create "${IMAGE_TAG}")
+trap 'docker rm -fv "$CID" >/dev/null' EXIT
+
+echo "[INFO] Copying '${ARTIFACT}' from container to host..."
+# The Dockerfile puts the artifact in the image root (/)
+if docker cp "${CID}:/${ARTIFACT}" "./${ARTIFACT}"; then
+    echo "[SUCCESS] Artifact '${ARTIFACT}' extracted to $(pwd)"
+else
+    echo "[ERROR] Failed to find '${ARTIFACT}' inside the image." >&2
+    exit 1
+fi 

binlog_json_parser/mysql_json_parser.cpp

@@ -120,24 +120,28 @@ static bool read_variable_length(const char *data, size_t data_length,
 
 std::string escape_json(const std::string &s) {
   std::ostringstream o;
-  for (auto c = s.cbegin(); c != s.cend(); c++) {
-    switch (*c) {
-      case '"': o << "\\\""; break;
+
+  for (const unsigned char uc : s) {
+    switch (uc) {
+      case '"':  o << "\\\""; break;
       case '\\': o << "\\\\"; break;
-      case '\b': o << "\\b"; break;
-      case '\f': o << "\\f"; break;
-      case '\n': o << "\\n"; break;
-      case '\r': o << "\\r"; break;
-      case '\t': o << "\\t"; break;
+      case '\b': o << "\\b";   break;
+      case '\f': o << "\\f";   break;
+      case '\n': o << "\\n";   break;
+      case '\r': o << "\\r";   break;
+      case '\t': o << "\\t";   break;
+
       default:
-        if (*c <= '\x1f') {
+        if (uc <= 0x1F) {
           o << "\\u"
-            << std::hex << std::setw(4) << std::setfill('0') << static_cast<int>(*c);
+            << std::hex << std::setw(4) << std::setfill('0') << static_cast<int>(uc)
+            << std::dec;
         } else {
-          o << *c;
+          o << static_cast<char>(uc);
         }
     }
   }
+
   return o.str();
 }