HPCC-31347 JLog trace export catch all Iexceptions

- Catches all IExceptions thrown in noexcept function - Logs issue - Releases exception - Reports export failure if exception caught Signed-off-by: Rodrigo Pastrana <[email protected]>
hpcc-systems · Mar 1, 2024 · 21bf908 · 21bf908
1 parent 2b8925b
commit 21bf908
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/system/jlib/jtrace.cpp b/system/jlib/jtrace.cpp
@@ -253,8 +253,10 @@ class JLogSpanExporter final : public opentelemetry::sdk::trace::SpanExporter
             }
             return opentelemetry::sdk::common::ExportResult::kSuccess;
         }
-        catch(...) //Interface does not allow throwing exceptions
+        catch (IException * e)
         {
+            EXCLOG(e, "JLogSpanExporter::Export");
+            e->Release();
             return opentelemetry::sdk::common::ExportResult::kFailure;
         }
     }