bug logging: remove double timestamp

commit_hash:ea7779bfcaaf15da3b45742aded8eeb7e03751fc

Author: segoon

core/src/logging/log_message_test.cpp

@@ -15,6 +15,7 @@
 #include <logging/rate_limit.hpp>
 #include <userver/decimal64/decimal64.hpp>
 #include <userver/formats/json/serialize.hpp>
+#include <userver/logging/impl/mem_logger.hpp>
 #include <userver/logging/null_logger.hpp>
 #include <userver/utils/regex.hpp>
 #include <userver/utils/traceful_exception.hpp>
@@ -93,6 +94,17 @@ TEST_F(LoggingTest, LogFormat) {
         << "Path shortening for logs stopped working.";
 }
 
+TEST_F(LoggingTest, MemLoggerLogFormat) {
+    logging::impl::MemLogger mem_logger;
+    mem_logger.ForwardTo(&*GetStreamLogger());
+    LOG_CRITICAL_TO(mem_logger) << "test" << logging::LogExtra{{"foo", "bar"}};
+    logging::LogFlush();
+
+    const auto str = GetStreamString();
+    constexpr std::string_view kTimestamp = "timestamp=";
+    EXPECT_EQ(str.find(kTimestamp), str.rfind(kTimestamp)) << str;
+}
+
 TEST_F(LoggingTest, FloatingPoint) {
     constexpr float f = 3.1415F;
     EXPECT_EQ(ToStringViaLogging(f), ToStringViaStreams(f));

universal/src/logging/impl/formatters/struct.cpp

@@ -14,12 +14,6 @@ namespace logging::impl::formatters {
 Struct::Struct(Level level, LogClass log_class) {
     item_.level = level;
     item_.log_class = log_class;
-
-    const auto now = std::chrono::system_clock::now();
-    item_.tags.emplace_back(
-        "timestamp",
-        fmt::format(FMT_COMPILE("{}.{:06}"), GetCurrentTimeString(now).ToStringView(), FractionalMicroseconds(now))
-    );
 }
 
 void Struct::AddTag(std::string_view key, const LogExtra::Value& value) { item_.tags.emplace_back(key, value); }