429 (Too Many Requests) is not unrecoverable error

cap-js · Aug 17, 2023 · 9b1134d · 9b1134d
1 parent ce38f93
commit 9b1134d
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/srv/log2restv2.js b/srv/log2restv2.js
@@ -22,8 +22,7 @@ module.exports = class AuditLog2RESTv2 extends AuditLogService {
       const { event, data } = req
 
       // event.match() is used to support the old event names
-      if (event === 'SensitiveDataRead' || event.match(/^dataAccess/i))
-        return this._handle(data, 'DATA_ACCESS')
+      if (event === 'SensitiveDataRead' || event.match(/^dataAccess/i)) return this._handle(data, 'DATA_ACCESS')
       if (event === 'PersonalDataModified' || event.match(/^dataModification/i))
         return this._handle(data, 'DATA_MODIFICATION')
       if (event === 'ConfigurationModified' || event.match(/^configChange/i))
@@ -65,7 +64,8 @@ module.exports = class AuditLog2RESTv2 extends AuditLogService {
       return access_token
     } catch (err) {
       // 401 could also mean x-zid is not valid
-      if (String(err.response?.statusCode).match(/^4\d\d$/)) err.unrecoverable = true
+      if (String(err.response?.statusCode).match(/^4\d\d$/) && err.response?.statusCode !== 429)
+        err.unrecoverable = true
       throw err
     }
   }