Merge branch 'release/1.0.6'

Author: vjain008

CHANGELOG.md

@@ -4,17 +4,25 @@ All notable changes to this project will be documented in this file. Dates are d
 
 Generated by [`auto-changelog`](https://github.com/CookPete/auto-changelog).
 
+#### [1.0.6](https://github.com/rdkcentral/rdkNativeScript/compare/1.0.5...1.0.6)
+
+- RDKEMW-8240 : Add logs in jsruntime.log in /opt/logs [`#60`](https://github.com/rdkcentral/rdkNativeScript/pull/60)
+- Merge tag '1.0.5' into develop [`0bae809`](https://github.com/rdkcentral/rdkNativeScript/commit/0bae809fc9bb8a69156798e07b15cade4c526431)
+
 #### [1.0.5](https://github.com/rdkcentral/rdkNativeScript/compare/1.0.4...1.0.5)
 
+> 15 September 2025
+
 - RDKEMW-8240 : Add logs in jsruntime.log in /opt/logs [`#57`](https://github.com/rdkcentral/rdkNativeScript/pull/57)
+- 1.0.5 release changelog updates [`bd46b16`](https://github.com/rdkcentral/rdkNativeScript/commit/bd46b16959b0b828e09154fdd5c8f8b72ea989cc)
 - Merge tag '1.0.4' into develop [`ad6b6ee`](https://github.com/rdkcentral/rdkNativeScript/commit/ad6b6ee37cb5c76239a29e10c053e054e90b25df)
 
 #### [1.0.4](https://github.com/rdkcentral/rdkNativeScript/compare/1.0.3...1.0.4)
 
 > 7 September 2025
 
 - RDKEMW-7489: Adding Href support [`#54`](https://github.com/rdkcentral/rdkNativeScript/pull/54)
-- RDKEMW-5556: Bug for terminateApplication [`#51`](https://github.com/rdkcentral/rdkNativeScript/pull/51)
+- Ticket:RDKEMW-5556: Bug for terminateApplication [`#51`](https://github.com/rdkcentral/rdkNativeScript/pull/51)
 - 1.0.4 release changelog updates [`2e56742`](https://github.com/rdkcentral/rdkNativeScript/commit/2e56742d0adda04fe61277fa0782e2b5b2dde19c)
 - Merge tag '1.0.3' into develop [`e15828d`](https://github.com/rdkcentral/rdkNativeScript/commit/e15828d5a330b072753e682f5c446b0f9ae14531)
 
@@ -37,9 +45,9 @@ Generated by [`auto-changelog`](https://github.com/CookPete/auto-changelog).
 - RDKEMW-4729:Add support for clearMeasures [`#33`](https://github.com/rdkcentral/rdkNativeScript/pull/33)
 - RDKEMW-3466 : JSRuntimeWidget [`#32`](https://github.com/rdkcentral/rdkNativeScript/pull/32)
 - RDKEMW-4353:Implement logs to remain [`#29`](https://github.com/rdkcentral/rdkNativeScript/pull/29)
-- RDKEMW-4353:Bug when set DEBUG level [`#28`](https://github.com/rdkcentral/rdkNativeScript/pull/28)
-- RDKEMW-4278:Update launchTime metrics [`#27`](https://github.com/rdkcentral/rdkNativeScript/pull/27)
-- RDKEMW-4353:Implement logs to remain [`32f6cf1`](https://github.com/rdkcentral/rdkNativeScript/commit/32f6cf11af9c786d34334838ff75065fbd0d70cf)
+- Ticket Number:RDKEMW-4353:Bug when set DEBUG level [`#28`](https://github.com/rdkcentral/rdkNativeScript/pull/28)
+- Ticket Number:RDKEMW-4278:Update launchTime metrics [`#27`](https://github.com/rdkcentral/rdkNativeScript/pull/27)
+- Ticket Number:RDKEMW-4353:Implement logs to remain [`32f6cf1`](https://github.com/rdkcentral/rdkNativeScript/commit/32f6cf11af9c786d34334838ff75065fbd0d70cf)
 - 1.0.2 release changelog updates [`bd89254`](https://github.com/rdkcentral/rdkNativeScript/commit/bd89254c510cbd92accda18c4df5fa8d7221615f)
 - RDKEMW-4729: Code cleanup for window change [`9162e1c`](https://github.com/rdkcentral/rdkNativeScript/commit/9162e1cc2b8c7a19c41a7420e19f05786df4899e)
 
@@ -49,12 +57,12 @@ Generated by [`auto-changelog`](https://github.com/CookPete/auto-changelog).
 
 - RDKEMW-3457 : WebSocket IPC [`#24`](https://github.com/rdkcentral/rdkNativeScript/pull/24)
 - RDKEMW-3523:Update rdkNativeScript with application identifiers [`#20`](https://github.com/rdkcentral/rdkNativeScript/pull/20)
-- RDKEMW-3430: Implementing log levels [`#22`](https://github.com/rdkcentral/rdkNativeScript/pull/22)
+- Ticket Number:RDKEMW-3430: Implementing log levels [`#22`](https://github.com/rdkcentral/rdkNativeScript/pull/22)
 - RDKEMW-3105 : Add ability to show network connections with details [`#18`](https://github.com/rdkcentral/rdkNativeScript/pull/18)
-- RDKEMW-3557:-Metrics for launchtime [`#19`](https://github.com/rdkcentral/rdkNativeScript/pull/19)
+- Ticket Number :RDKEMW-3557:-Metrics for launchtime [`#19`](https://github.com/rdkcentral/rdkNativeScript/pull/19)
 - RDKEMW-2175 : Run player within jsruntime plugin via dynamic load of … [`#16`](https://github.com/rdkcentral/rdkNativeScript/pull/16)
-- RDKEMW-3523:Update rdkNativeScript with interface API's [`60fc51b`](https://github.com/rdkcentral/rdkNativeScript/commit/60fc51b6f48272e9671bc787c68a2ce19b036e0c)
-- RDKEMW-3557:Metrics for launch time [`eca1c15`](https://github.com/rdkcentral/rdkNativeScript/commit/eca1c15b2e98e40156dba22696325c2c5d5fbaa5)
+- 3523:Update rdkNativeScript with interface API's [`60fc51b`](https://github.com/rdkcentral/rdkNativeScript/commit/60fc51b6f48272e9671bc787c68a2ce19b036e0c)
+- Ticket Number:RDKEMW-3557:Metrics for launch time [`eca1c15`](https://github.com/rdkcentral/rdkNativeScript/commit/eca1c15b2e98e40156dba22696325c2c5d5fbaa5)
 - 1.0.1 release changelog updates [`e60bbb2`](https://github.com/rdkcentral/rdkNativeScript/commit/e60bbb2b72c74a43c34af35179d4b619fd9ac779)
 
 #### 1.0.0

CMakeLists.txt

@@ -38,6 +38,7 @@ option(ENABLE_AAMP_JSBINDINGS_DYNAMIC "ENABLE_AAMP_JSBINDINGS_DYNAMIC" OFF)
 option(ENABLE_JSRUNTIME_PLAYER "ENABLE_JSRUNTIME_PLAYER" OFF)
 option(ENABLE_JSRUNTIME_THUNDER_SECURITYAGENT "ENABLE_JSRUNTIME_THUNDER_SECURITYAGENT" OFF)
 option(BUILD_JSRUNTIME_DESKTOP "BUILD_JSRUNTIME_DESKTOP" ON)
+option(USE_ETHANLOG "USE_ETHANLOG" OFF)
 option(PKG_CONFIG_SYSROOT_DIR "PKG_CONFIG_SYSROOT_DIR" "${CMAKE_CURRENT_SOURCE_DIR}/externals/extlibs")
 
 if(JSRUNTIME_ENGINE_NAME STREQUAL "jsc")
@@ -158,21 +159,27 @@ else ()
     set(JSRUNTIME_LIBRARY_LINK_DIRECTORIES ${JSRUNTIME_LIBRARY_LINK_DIRECTORIES} -L${CMAKE_CURRENT_SOURCE_DIR}/../build/)
 endif (BUILD_JSRUNTIME_DESKTOP)
 
+if (USE_ETHANLOG)
+	message("Enabling Ethan logs")
+        add_definitions("-DUSE_ETHANLOG")
+	set(JSRUNTIME_LINK_ETHANLIB -lethanlog)
+endif (USE_ETHANLOG)
+
 if (BUILD_JSRUNTIME_APP)
     message("Enabling build support for jsruntime app")
     add_executable(jsruntime_app ${JSRUNTIME_APP_FILES})
     add_dependencies(${JSRUNTIME_LIBRARY_NAME} ${JSRUNTIME_ENGINE_DEPENDENT_LIBRARY_NAME})
     add_dependencies(jsruntime_app ${JSRUNTIME_LIBRARY_NAME} ${JSRUNTIME_ENGINE_DEPENDENT_LIBRARY_NAME} )
     target_include_directories(jsruntime_app PRIVATE ${JSRUNTIME_INCLUDE_DIRECTORIES})
     set_target_properties(jsruntime_app PROPERTIES OUTPUT_NAME ${EXE})
-    target_link_libraries(jsruntime_app ${JSRUNTIME_LIBRARY_LINK_DIRECTORIES} -l${JSRUNTIME_LIBRARY_NAME} ${JSRUNTIME_LINK_LIBRARIES})
+    target_link_libraries(jsruntime_app ${JSRUNTIME_LIBRARY_LINK_DIRECTORIES} -l${JSRUNTIME_LIBRARY_NAME} ${JSRUNTIME_LINK_LIBRARIES} ${JSRUNTIME_LINK_ETHANLIB})
 endif (BUILD_JSRUNTIME_APP)
 
 if (BUILD_JSRUNTIME_CLIENT)
     add_executable(jsruntime_client ${JSRUNTIME_CLIENT_FILES})
     target_include_directories(jsruntime_client PRIVATE ${JSRUNTIME_INCLUDE_DIRECTORIES})
     set_target_properties(jsruntime_client PROPERTIES OUTPUT_NAME "JSRuntimeClient")
-    target_link_libraries(jsruntime_client ${JSRUNTIME_LIBRARY_LINK_DIRECTORIES} -lpthread)
+    target_link_libraries(jsruntime_client ${JSRUNTIME_LIBRARY_LINK_DIRECTORIES} ${JSRUNTIME_LINK_ETHANLIB} -lpthread)
 endif (BUILD_JSRUNTIME_CLIENT)
 
 set(UWEBSOCKETS_TARGET "Linux")

include/NativeJSLogger.h

@@ -32,9 +32,11 @@ class NativeJSLogger
 {
     public:
         static void setLogLevel(const char* loglevel);
+	static void isEthanLogEnabled();
         static void log(LogLevel level, const char* format, ...);
     private:
         static LogLevel sLogLevel;
+	static bool mEthanLogEnabled;
 };
 
 #endif // NATIVEJS_LOGGER_H

src/NativeJSLogger.cpp

@@ -24,13 +24,27 @@
 #include <unistd.h>
 #include <sys/syscall.h>
 #include <stdlib.h>
-#include <time.h>
+
+#ifdef USE_ETHANLOG
+#include <ethanlog.h>
+
+static const int ethanLogLevelMap[] = {
+    ETHAN_LOG_DEBUG, ETHAN_LOG_INFO, ETHAN_LOG_WARNING, ETHAN_LOG_ERROR, ETHAN_LOG_FATAL   
+};
+#endif
 
 static const char* logLevelNames[] = {
         "DEBUG", "INFO", "WARN", "ERROR", "FATAL"
 };
 
+
 LogLevel NativeJSLogger::sLogLevel = INFO;
+bool NativeJSLogger::mEthanLogEnabled = false;
+
+void NativeJSLogger::isEthanLogEnabled()
+{
+    mEthanLogEnabled  = (getenv("ETHAN_LOGGING_PIPE") != nullptr);
+}
 
 void NativeJSLogger::setLogLevel(const char* loglevel)
 {
@@ -50,35 +64,36 @@ void NativeJSLogger::setLogLevel(const char* loglevel)
 
 void NativeJSLogger::log(LogLevel level, const char* format, ...)
 {
-     if (level < sLogLevel)
-            return;
-
-      FILE* logFile = fopen("/opt/logs/jsruntime.log", "a");
-      if (!logFile) {
-            perror("Failed to open jsruntime.log");
-            return;
-      }
+    if (level < sLogLevel)
+        return;
 
-      char timeStr[64];
-      time_t now = time(NULL);
-      struct tm* t = localtime(&now);
-      strftime(timeStr, sizeof(timeStr), "%Y-%m-%d %H:%M:%S", t);
+    va_list args;
+    va_start(args, format);
+	int threadId = syscall(__NR_gettid);
 
-      int threadId = syscall(__NR_gettid);
-      const char* levelStr = logLevelNames[level];
-      char buffer[512];
+#ifdef USE_ETHANLOG
+    if (mEthanLogEnabled)
+    {
+	int ethanLogLevel = ethanLogLevelMap[level];
+        char* newFormat = nullptr;
+        int len = snprintf(nullptr, 0, "JSRuntime [Thread-%d] %s", threadId, format);
+        if (len > 0) {
+            newFormat = (char*)alloca(len + 1);
+            snprintf(newFormat, len + 1, "JSRuntime [Thread-%d] %s", threadId, format);
+        } else {
+            newFormat = (char*)format;
+        }
 
-      va_list args;
-      va_start(args, format);
-      vsnprintf(buffer, sizeof(buffer), format, args);
-      va_end(args);
+        vethanlog(ethanLogLevel, "JsRuntime", nullptr, -1, newFormat, args);
+    }
+#else
+    
+    const char* levelStr = logLevelNames[level];
+    char buffer[512];
+    vsnprintf(buffer, sizeof(buffer), format, args);
+    printf("\n[%s] JsRuntime Thread-%d: %s\n", levelStr, threadId, buffer);
 
-      if(getenv("NATIVEJS_LOGS_REG"))
-      	 printf("\n[%s] JsRuntime Thread-%d: %s\n", levelStr, threadId, buffer);
-      else
-        fprintf(logFile, "%s : [%s] JsRuntime Thread-%d: %s\n", timeStr, levelStr, threadId, buffer);
-
-      fclose(logFile);
+#endif //USE_ETHANLOG    
 
+    va_end(args);
 }
-

src/NativeJSRenderer.cpp

@@ -103,6 +103,11 @@ NativeJSRenderer::NativeJSRenderer(std::string waylandDisplay): mEngine(nullptr)
     }
 
     const char* levelFromEnv = getenv("NATIVEJS_LOG_LEVEL");
+    // checking for ethan log env 
+    #ifdef USE_ETHANLOG
+    NativeJSLogger::isEthanLogEnabled();
+    NativeJSLogger::log(INFO, "EthanLog is enabled");
+    #endif
     if(levelFromEnv)
     {
     	NativeJSLogger::setLogLevel(levelFromEnv);