IR-2915: Re-evaluate feathers validator usage for recording services (#…

…10545) * Fixed validators * chore: add createdAt to recordingQueryProperties
EtherealEngine · Aug 16, 2024 · 09a2269 · 09a2269
1 parent 85aaea5
commit 09a2269
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 8 deletions.
diff --git a/packages/common/src/schemas/recording/recording.schema.ts b/packages/common/src/schemas/recording/recording.schema.ts
@@ -69,7 +69,7 @@ export interface RecordingDatabaseType extends Omit<RecordingType, 'schema'> {
 }
 
 // Schema for creating new entries
-export const recordingDataSchema = Type.Partial(recordingSchema, {
+export const recordingDataSchema = Type.Pick(recordingSchema, ['schema'], {
   $id: 'RecordingData'
 })
 export interface RecordingData extends Static<typeof recordingDataSchema> {}
@@ -81,7 +81,7 @@ export const recordingPatchSchema = Type.Partial(recordingSchema, {
 export interface RecordingPatch extends Static<typeof recordingPatchSchema> {}
 
 // Schema for allowed query properties
-export const recordingQueryProperties = Type.Pick(recordingSchema, ['id', 'userId'])
+export const recordingQueryProperties = Type.Pick(recordingSchema, ['id', 'userId', 'createdAt'])
 export const recordingQuerySchema = Type.Intersect(
   [
     querySyntax(recordingQueryProperties),

diff --git a/packages/server-core/src/recording/recording-resource/recording-resource.hooks.ts b/packages/server-core/src/recording/recording-resource/recording-resource.hooks.ts
@@ -50,20 +50,20 @@ export default {
 
   before: {
     all: [
-      () => schemaHooks.validateQuery(recordingResourceQueryValidator),
+      schemaHooks.validateQuery(recordingResourceQueryValidator),
       schemaHooks.resolveQuery(recordingResourceQueryResolver)
     ],
     find: [iff(isProvider('external'), verifyScope('recording', 'read'))],
     get: [iff(isProvider('external'), verifyScope('recording', 'read'))],
     create: [
       iff(isProvider('external'), verifyScope('recording', 'write'), verifyScope('settings', 'write')),
-      () => schemaHooks.validateData(recordingResourceDataValidator),
+      schemaHooks.validateData(recordingResourceDataValidator),
       schemaHooks.resolveData(recordingResourceDataResolver)
     ],
     update: [disallow()],
     patch: [
       iff(isProvider('external'), verifyScope('recording', 'write')),
-      () => schemaHooks.validateData(recordingResourcePatchValidator),
+      schemaHooks.validateData(recordingResourcePatchValidator),
       schemaHooks.resolveData(recordingResourcePatchResolver)
     ],
     remove: [iff(isProvider('external'), verifyScope('recording', 'write'))]

diff --git a/packages/server-core/src/recording/recording/recording.hooks.ts b/packages/server-core/src/recording/recording/recording.hooks.ts
@@ -87,7 +87,7 @@ export default {
   },
 
   before: {
-    all: [() => schemaHooks.validateQuery(recordingQueryValidator), schemaHooks.resolveQuery(recordingQueryResolver)],
+    all: [schemaHooks.validateQuery(recordingQueryValidator), schemaHooks.resolveQuery(recordingQueryResolver)],
     find: [
       iff(
         isProvider('external'),
@@ -100,13 +100,13 @@ export default {
     create: [
       iff(isProvider('external'), verifyScope('recording', 'write')),
       setLoggedinUserInBody('userId'),
-      () => schemaHooks.validateData(recordingDataValidator),
+      schemaHooks.validateData(recordingDataValidator),
       schemaHooks.resolveData(recordingDataResolver)
     ],
     update: [disallow()],
     patch: [
       iff(isProvider('external'), verifyScope('recording', 'write')),
-      () => schemaHooks.validateData(recordingPatchValidator),
+      schemaHooks.validateData(recordingPatchValidator),
       schemaHooks.resolveData(recordingPatchResolver)
     ],
     remove: [iff(isProvider('external'), verifyScope('recording', 'write')), ensureRecording]