TNO-3062 minor updates to the logs (#2283)

- should re-throw exception after logging
bcgov · Sep 12, 2024 · 2ad65bf · 2ad65bf
1 parent 73eea58
commit 2ad65bf
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 0 deletions.
diff --git a/libs/net/dal/Services/ProductService.cs b/libs/net/dal/Services/ProductService.cs
@@ -444,6 +444,7 @@ public void AddAndSave(UserProduct subscription)
         } catch (Exception ex)
         {
             this.Logger.LogError(ex, $"ProductService - AddAndSave ProductId: {subscription.ProductId}, UserId: {subscription.UserId} throws exception.");
+            throw;
         }
     }
 
@@ -460,6 +461,7 @@ public void UpdateAndSave(UserProduct subscription)
         } catch (Exception ex)
         {
             this.Logger.LogError(ex, $"ProductService - UpdateAndSave ProductId: {subscription.ProductId}, UserId: {subscription.UserId} throws exception.");
+            throw;
         }
     }
 
@@ -494,6 +496,7 @@ public UserProduct Unsubscribe(int userId, int productId)
         } catch (Exception ex)
         {
             this.Logger.LogError(ex, $"ProductService - Unsubscribe userId: {userId}, productId: {productId} throws exception.");
+            throw;
         }
         return userProduct;
     }
@@ -529,6 +532,7 @@ public UserProduct Subscribe(int userId, int productId)
         } catch (Exception ex)
         {
             this.Logger.LogError(ex, $"ProductService - Subscribe userId: {userId}, productId: {productId} throws exception.");
+            throw;
         }
         return userProduct;
     }

diff --git a/libs/net/dal/Services/ReportService.cs b/libs/net/dal/Services/ReportService.cs
@@ -1094,6 +1094,7 @@ public IEnumerable<long> GetRelatedReportInstanceContentToExclude(int reportId)
         } catch (Exception ex)
         {
             this.Logger.LogError(ex, $"ReportService - ClearFoldersInReport Report Id: {report.Id} throws exception.");
+            throw;
         }
 
         return FindById(report.Id);

diff --git a/services/net/filemonitor/FilemonitorAction.cs b/services/net/filemonitor/FilemonitorAction.cs
@@ -735,6 +735,7 @@ private string FixBlacksNewsgroupXml(string filePath, string xml)
                 } catch (Exception ex)
                 {
                     this.Logger.LogError(ex, $"FixBlacksNewsgroupXml - process XML throws exceptions. The XML is: {xml}");
+                    throw;
                 }
             }