fix(logging): don't unconditionally log to console (#4136)

getsentry · Jul 10, 2024 · e773cad · e773cad
1 parent 152437a
commit e773cad
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 7 deletions.
diff --git a/Sources/Sentry/SentryAsyncSafeLog.c b/Sources/Sentry/SentryAsyncSafeLog.c
@@ -91,7 +91,6 @@ writeToLog(const char *const str)
  pos += bytesWritten;
  }
  }
- write(STDOUT_FILENO, str, strlen(str));
 
 #if SENTRY_ASYNC_SAFE_LOG_ALSO_WRITE_TO_CONSOLE
  // if we're debugging, also write the log statements to the console; we only check once for
@@ -150,10 +149,10 @@ sentry_asyncLogSetFileName(const char *filename, bool overwrite)
 }
 
 void
-sentry_asyncLogC(const char *const level, const char *const file, const int line,
- const char *const function, const char *const fmt, ...)
+sentry_asyncLogC(
+ const char *const level, const char *const file, const int line, const char *const fmt, ...)
 {
- writeFmtToLog("%s: %s (%u): %s: ", level, lastPathEntry(file), line, function);
+ writeFmtToLog("%s: %s (%u): ", level, lastPathEntry(file), line);
  va_list args;
  va_start(args, fmt);
  writeFmtArgsToLog(fmt, args);

diff --git a/Sources/Sentry/SentryAsyncSafeLog.h b/Sources/Sentry/SentryAsyncSafeLog.h
@@ -47,8 +47,7 @@ extern "C" {
 
 static char g_logFilename[1024];
 
-void sentry_asyncLogC(
- const char *level, const char *file, int line, const char *function, const char *fmt, ...);
+void sentry_asyncLogC(const char *level, const char *file, int line, const char *fmt, ...);
 
 #define i_SENTRY_ASYNC_SAFE_LOG sentry_asyncLogC
 
@@ -62,7 +61,7 @@ void sentry_asyncLogC(
 #define SENTRY_ASYNC_SAFE_LOG_LEVEL SENTRY_ASYNC_SAFE_LOG_LEVEL_ERROR
 
 #define a_SENTRY_ASYNC_SAFE_LOG(LEVEL, FMT, ...) \
- i_SENTRY_ASYNC_SAFE_LOG(LEVEL, __FILE__, __LINE__, __PRETTY_FUNCTION__, FMT, ##__VA_ARGS__)
+ i_SENTRY_ASYNC_SAFE_LOG(LEVEL, __FILE__, __LINE__, FMT, ##__VA_ARGS__)
 
 // ============================================================================
 #pragma mark - API -