Narrower type for expiration

PROCEED-Labs · Oct 23, 2023 · c6b961f · c6b961f
1 parent c69a17a
commit c6b961f
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/management-system/src/backend/server/iam/rest-api/roles.js b/src/management-system/src/backend/server/iam/rest-api/roles.js
@@ -88,7 +88,7 @@ rolesRouter.put('/:id', validateRole, isAllowed('update', 'Role'), async (req, r
     try {
       // validateRole turns expiration into a Date, in order for the object merge
       // to work, we need it to be a string (type safe option in mergeIntoObject)
-      if (typeof role.expiration === 'object') {
+      if (role.expiration !== null && typeof role.expiration === 'object') {
         /** @type {Date} */
         const expirationDate = role.expiration;
         role.expiration = expirationDate.toISOString();