Added more logging when rate limiting is hit

DFE-Digital · Dec 19, 2024 · 841f382 · 841f382
1 parent ac1b899
commit 841f382
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/TeachingRecordSystem/src/TeachingRecordSystem.Core/Services/TrsDataSync/TrsDataSyncHelper.cs b/TeachingRecordSystem/src/TeachingRecordSystem.Core/Services/TrsDataSync/TrsDataSyncHelper.cs
@@ -832,6 +832,7 @@ private async Task<IReadOnlyDictionary<Guid, AuditDetailCollection>> GetAuditRec
                     }
                     catch (FaultException fex) when (fex.IsCrmRateLimitException(out var retryAfter))
                     {
+                        _logger.LogWarning("Hit CRM service limits; Fault exception");
                         await Task.Delay(retryAfter, cancellationToken);
                         continue;
                     }
@@ -842,11 +843,13 @@ private async Task<IReadOnlyDictionary<Guid, AuditDetailCollection>> GetAuditRec
 
                         if (firstFault.IsCrmRateLimitFault(out var retryAfter))
                         {
+                            _logger.LogWarning("Hit CRM service limits; CRM rate limit fault");
                             await Task.Delay(retryAfter, cancellationToken);
                             continue;
                         }
                         else if (firstFault.Message.Contains("The HTTP status code of the response was not expected (429)"))
                         {
+                            _logger.LogWarning("Hit CRM service limits; 429 too many requests");
                             await Task.Delay(TimeSpan.FromMinutes(2), cancellationToken);
                             continue;
                         }

diff --git a/TeachingRecordSystem/src/TeachingRecordSystem.Worker/appsettings.Production.json b/TeachingRecordSystem/src/TeachingRecordSystem.Worker/appsettings.Production.json
@@ -3,7 +3,9 @@
     "MinimumLevel": {
       "Default": "Error",
       "Override": {
-        "TeachingRecordSystem.Worker": "Warning"
+        "TeachingRecordSystem.Worker": "Warning",
+        "TeachingRecordSystem.Core.Services.TrsDataSync": "Warning",
+        "TeachingRecordSystem.Core.Jobs": "Warning"
       }
     }
   },