Convert item field in validateItemAccess to boolean (directus#24026)

VincentKempers · Nov 12, 2024 · 9e0bec9 · 9e0bec9
1 parent d0c7491
commit 9e0bec9
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/.changeset/proud-deers-kneel.md b/.changeset/proud-deers-kneel.md
@@ -0,0 +1,5 @@
+---
+'@directus/api': patch
+---
+
+Fixed an issue in the item access validation that caused problems with CockroachDB
diff --git a/api/src/permissions/modules/validate-access/lib/validate-item-access.ts b/api/src/permissions/modules/validate-access/lib/validate-item-access.ts
@@ -1,4 +1,5 @@
 import type { Accountability, PermissionsAction, PrimaryKey } from '@directus/types';
+import { toBoolean } from '@directus/utils';
 import { fetchPermittedAstRootFields } from '../../../../database/run-ast/modules/fetch-permitted-ast-root-fields.js';
 import type { AST } from '../../../../types/index.js';
 import type { Context } from '../../../types.js';
@@ -51,7 +52,7 @@ export async function validateItemAccess(options: ValidateItemAccessOptions, con
 		const { fields } = options;
 
 		if (fields) {
-			return items.every((item: any) => fields.every((field) => item[field] === 1));
+			return items.every((item: any) => fields.every((field) => toBoolean(item[field])));
 		}
 
 		return true;