fix(api-filters): Return valid action.mailbox value for a filter object

nodemailer · Oct 16, 2023 · c00cc02 · c00cc02
1 parent 292bbc2
commit c00cc02
Show file tree

Hide file tree

Showing 4 changed files with 330 additions and 291 deletions.
diff --git a/lib/api/filters.js b/lib/api/filters.js
@@ -405,8 +405,7 @@ module.exports = (db, server, userHandler, settingsHandler) => {
                 query: {},
                 action: {},
                 disabled: !!filterData.disabled,
-                created: filterData.created,
-                _filterData: filterData
+                created: filterData.created
             };
 
             Object.keys((filterData.query && filterData.query.headers) || {}).forEach(key => {
@@ -424,7 +423,15 @@ module.exports = (db, server, userHandler, settingsHandler) => {
                     response.action.targets = filterData.action.targets.map(target => target.value);
                     return;
                 }
-                response.action[key] = filterData.action[key];
+
+                switch (key) {
+                    case 'mailbox':
+                        // cast ObjectId value to a string, otherwise `permission.filter` will mess up the value
+                        response.action[key] = filterData.action[key].toString();
+                        break;
+                    default:
+                        response.action[key] = filterData.action[key];
+                }
             });
 
             if (filterData.metaData) {