refactor: optimize JSON handling, fix pointer safety, and improve overall stability

[lozatto]

This PR introduces several stability improvements, memory optimizations, and crash preventions across the LogApi module. The main focus was to replace pass-by-value with constant references for complex JSON objects, prevent potential crashes from unvalidated engine strings/pointers, and fix an issue with malformed UTF-8 data serialization.

🛠️ Key Changes

• Performance & Memory Optimization:

• Changed nlohmann::ordered_json parameters from pass-by-value to pass-by-const-reference (const nlohmann::ordered_json&) across multiple functions (SendEvent, EventResult, ClientPrint, etc.) to eliminate redundant deep copies.

• Crash & Exploit Prevention:

• Added nlohmann::ordered_json::error_handler_t::replace during Event.dump() serialization to gracefully handle and replace invalid UTF-8 characters, avoiding potential server crashes.

• Wrapped critical engine interactions and string conversions inside try-catch blocks (std::exception) to catch generic failures beyond just JSON parse errors.

• Added strict null-pointer and validation checks for game engine entities and strings (pvPrivateData, net_address, hostname, mapname).

• Bug Fixes & Refactoring:

• Fixed a bug in CLogPlayer::GetPlayerJson by copying the rawAuth engine pointer immediately into a std::string, preventing it from being invalidated by subsequent engine calls.

• Improved buffer safety in menu handling (LogMenu.cpp) by enforcing explicit limits (MAX_BUFFER_MENU) and manual null-termination.

• Tuned Makefile flags: added -fno-strict-aliasing and removed -static-libgcc -static-libstdc++ from the linker to resolve potential binary compatibility issues on Linux.

[lozatto]

@SmileYzn , I’m not sure if you had the opportunity to review the PR.