changed ZEN_ERROR to ZEN_WARN to reduce Sentry noise

addresses ZEN-SERVER-W5. This should not be logged as an error since the failure is propagated to the client and the failure is typically due to invalid input

1 files changed, 8 insertions, 3 deletions

diff --git a/src/zenserver/projectstore/remoteprojectstore.cpp b/src/zenserver/projectstore/remoteprojectstore.cpp
index c466b1cbb..c0fbad755 100644
--- a/src/zenserver/projectstore/remoteprojectstore.cpp
+++ b/src/zenserver/projectstore/remoteprojectstore.cpp
@@ -538,20 +538,25 @@ SaveOplog(CidStore&			   ChunkStore,
 												   OnBlock,
 												   OnLargeAttachment,
 												   OnBlockChunks,
-												   RemoteResult);
+												   /* out */ RemoteResult);
 
 	if (!RemoteResult.IsError())
 	{
 		uint64_t ChunkCount = OplogContainerObject["chunks"sv].AsArrayView().Num();
 		uint64_t BlockCount = OplogContainerObject["blocks"sv].AsArrayView().Num();
 		ZEN_INFO("Saving oplog container with {} attachments and {} blocks...", ChunkCount, BlockCount);
+
 		RemoteProjectStore::SaveResult ContainerSaveResult = RemoteStore.SaveContainer(OplogContainerObject.GetBuffer().AsIoBuffer());
 		if (ContainerSaveResult.ErrorCode)
 		{
 			RemoteResult.SetError(ContainerSaveResult.ErrorCode, ContainerSaveResult.Reason, "Failed to save oplog container");
-			ZEN_ERROR("Failed to save oplog container ({}). Reason: '{}'", RemoteResult.GetErrorReason(), RemoteResult.GetError());
+			ZEN_WARN("Failed to save oplog container ({}). Reason: '{}'", RemoteResult.GetErrorReason(), RemoteResult.GetError());
+		}
+		else
+		{
+			ZEN_DEBUG("Saved container in {}", NiceTimeSpanMs(static_cast<uint64_t>(ContainerSaveResult.ElapsedSeconds * 1000)));
 		}
-		ZEN_DEBUG("Saved container in {}", NiceTimeSpanMs(static_cast<uint64_t>(ContainerSaveResult.ElapsedSeconds * 1000)));
+
 		if (!ContainerSaveResult.Needs.empty())
 		{
 			ZEN_INFO("Filtering needed attachments...");