Просмотр исходного кода

[-] chore(logging): keep per-action tracing to debug

Review: not a review point. Raised because the added logging made an
ordinary session noisy to read.
Resolution: per-action outcomes move to debug; the boot-time extraction
counts stay at info.

Seven per-action outcomes logged at info: character selection, equip,
both socket paths, item state, dismantle and the Collections pull. Info
is where upstream records boot and lifecycle milestones, so an action
the player takes repeatedly does not belong there, and every one of them
fired on a normal session. They move to debug, beside the pipeline trace
they already belonged to.

The extraction counts stay at info. Those report once per boot and are
how a bad content build is spotted.

Nothing else changes: format and failure levels are untouched.

Thomas Shields 3 недель назад
Родитель
Сommit
81f9f31df9
1 измененных файлов с 7 добавлено и 7 удалено
  1. 7 7
      Sunrise/src/server/web_service/web_service_actions.cpp

+ 7 - 7
Sunrise/src/server/web_service/web_service_actions.cpp

@@ -223,7 +223,7 @@ void select_character(const middleware::web_service::Message& message, Outcome&
                                       static_cast<unsigned>(changed));
                                       static_cast<unsigned>(changed));
     if (written > 0) {
     if (written > 0) {
         core::log::write(core::log::Channel::server,
         core::log::write(core::log::Channel::server,
-                         core::log::Level::info,
+                         core::log::Level::debug,
                          {line.data(), static_cast<std::size_t>(written)});
                          {line.data(), static_cast<std::size_t>(written)});
     }
     }
 }
 }
@@ -295,7 +295,7 @@ void mutate_equipment(const middleware::web_service::Message& message,
                       mutation.movedItemCount);
                       mutation.movedItemCount);
     if (count > 0) {
     if (count > 0) {
         core::log::write(core::log::Channel::server,
         core::log::write(core::log::Channel::server,
-                         core::log::Level::info,
+                         core::log::Level::debug,
                          {line.data(), static_cast<std::size_t>(count)});
                          {line.data(), static_cast<std::size_t>(count)});
     }
     }
 }
 }
@@ -372,7 +372,7 @@ void mutate_socket_plug(const middleware::web_service::Message& message,
         mutation.itemIndex);
         mutation.itemIndex);
     if (count > 0) {
     if (count > 0) {
         core::log::write(core::log::Channel::server,
         core::log::write(core::log::Channel::server,
-                         core::log::Level::info,
+                         core::log::Level::debug,
                          {line.data(), static_cast<std::size_t>(count)});
                          {line.data(), static_cast<std::size_t>(count)});
     }
     }
 }
 }
@@ -470,7 +470,7 @@ void mutate_equipped_socket_plug(const middleware::web_service::Message& message
         static_cast<unsigned long long>(replacement.auxiliary));
         static_cast<unsigned long long>(replacement.auxiliary));
     if (count > 0) {
     if (count > 0) {
         core::log::write(core::log::Channel::server,
         core::log::write(core::log::Channel::server,
-                         core::log::Level::info,
+                         core::log::Level::debug,
                          {line.data(), static_cast<std::size_t>(count)});
                          {line.data(), static_cast<std::size_t>(count)});
     }
     }
 }
 }
@@ -521,7 +521,7 @@ void mutate_item_state(const middleware::web_service::Message& message, Outcome&
         mutation.itemIndex);
         mutation.itemIndex);
     if (count > 0) {
     if (count > 0) {
         core::log::write(core::log::Channel::server,
         core::log::write(core::log::Channel::server,
-                         core::log::Level::info,
+                         core::log::Level::debug,
                          {line.data(), static_cast<std::size_t>(count)});
                          {line.data(), static_cast<std::size_t>(count)});
     }
     }
 }
 }
@@ -552,7 +552,7 @@ void report_item_dismantle(const middleware::web_service::Message& message,
         quantity);
         quantity);
     if (count > 0) {
     if (count > 0) {
         core::log::write(core::log::Channel::server,
         core::log::write(core::log::Channel::server,
-                         result == "ok" ? core::log::Level::info : core::log::Level::warn,
+                         result == "ok" ? core::log::Level::debug : core::log::Level::warn,
                          {line.data(), static_cast<std::size_t>(count)});
                          {line.data(), static_cast<std::size_t>(count)});
     }
     }
 }
 }
@@ -639,7 +639,7 @@ void report_item_acquisition(const middleware::web_service::Message& message,
         static_cast<unsigned long long>(instanceSoid));
         static_cast<unsigned long long>(instanceSoid));
     if (count > 0) {
     if (count > 0) {
         core::log::write(core::log::Channel::server,
         core::log::write(core::log::Channel::server,
-                         result == "ok" ? core::log::Level::info : core::log::Level::warn,
+                         result == "ok" ? core::log::Level::debug : core::log::Level::warn,
                          {line.data(), static_cast<std::size_t>(count)});
                          {line.data(), static_cast<std::size_t>(count)});
     }
     }
 }
 }