Add extra logging to debug error when page changes

jamerst · Oct 30, 2022 · 8896ef7 · 8896ef7
1 parent fe2f72d
commit 8896ef7
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 5 deletions.
diff --git a/jobhunt/Data/JobHuntContext.cs b/jobhunt/Data/JobHuntContext.cs
@@ -17,7 +17,8 @@ protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder) {
         // });
 
         // optionsBuilder.UseLoggerFactory(loggerFactory);
-        // optionsBuilder.EnableSensitiveDataLogging();
+
+        optionsBuilder.EnableSensitiveDataLogging();
 
         // disable global query filter warning for related entries
         // without this a warning is thrown at every start for the JobCategories entity

diff --git a/jobhunt/PageWatcher/PageWatcher.cs b/jobhunt/PageWatcher/PageWatcher.cs
@@ -163,12 +163,21 @@ await _wpcService.CreateAsync(new WatchedPageChange
         {
             changed = true;
 
-            WatchedPageChange? change = await _wpcService.CreateAsync(new WatchedPageChange
+            WatchedPageChange change = new WatchedPageChange
             {
                 WatchedPageId = page.Id,
                 Created = DateTimeOffset.UtcNow,
                 Html = response
-            });
+            };
+
+            try
+            {
+                await _wpcService.CreateAsync(change);
+            }
+            catch (Exception ex)
+            {
+                _logger.LogError(ex, "Failed to save page change {@change}", change);
+            }
 
             if (change != null)
             {

diff --git a/jobhunt/Services/BaseServices/BaseService.cs b/jobhunt/Services/BaseServices/BaseService.cs
@@ -14,7 +14,7 @@ public BaseService(JobHuntContext context)
 
     public Task<int> SaveChangesAsync() => _context.SaveChangesAsync();
 
-    public virtual async Task<T?> CreateAsync(T entity)
+    public virtual async Task<T> CreateAsync(T entity)
     {
         Set.Add(entity);
 
@@ -35,6 +35,6 @@ public interface IBaseService<T> where T : class, KeyedEntity
 {
     DbSet<T> Set { get; }
     Task<int> SaveChangesAsync();
-    Task<T?> CreateAsync(T entity);
+    Task<T> CreateAsync(T entity);
     Task<T> BeforeSaveAsync(T entity);
 }