Test

Author: MarceloRGonc

packages/server/api/src/app/flow-template/cloud-template.controller.ts

@@ -4,9 +4,7 @@ import {
 } from '@fastify/type-provider-typebox';
 import { AppSystemProp, logger, system } from '@openops/server-shared';
 import { ALL_PRINCIPAL_TYPES, OpenOpsId } from '@openops/shared';
-import {
-  allowAllOriginsHookHandlerTest,
-} from '../helper/allow-all-origins-hook-handler';
+import { allowAllOriginsHookHandler } from '../helper/allow-all-origins-hook-handler';
 import { getVerifiedUser } from '../user-info/cloud-auth';
 import { flowTemplateService } from './flow-template.service';
 
@@ -25,19 +23,17 @@ export const cloudTemplateController: FastifyPluginAsyncTypebox = async (
   //   return;
   // }
 
-  // app.addHook('onRequest', allowAllOriginsHookHandler);
-
   // cloud templates are available on any origin
   app.options(
-    '/*',
+    '*',
     {
       config: {
         allowedPrincipals: ALL_PRINCIPAL_TYPES,
         skipAuth: true,
         cors: false,
       },
     },
-    allowAllOriginsHookHandlerTest,
+    allowAllOriginsHookHandler,
   );
 
   app.get(

packages/server/api/src/app/helper/allow-all-origins-hook-handler.ts

@@ -1,35 +1,7 @@
 import { logger } from '@openops/server-shared';
 import { FastifyReply, FastifyRequest } from 'fastify';
-import { onRequestHookHandler } from 'fastify/types/hooks';
 
-export const allowAllOriginsHookHandler: onRequestHookHandler = (
-  request,
-  reply,
-  done,
-) => {
-  void reply.header(
-    'Access-Control-Allow-Origin',
-    request.headers.origin || request.headers['Ops-Origin'] || '*',
-  );
-
-  void reply.header('Access-Control-Allow-Methods', 'GET,OPTIONS');
-
-  void reply.header(
-    'Access-Control-Allow-Headers',
-    'Content-Type,Ops-Origin,Authorization',
-  );
-
-  void reply.header('Access-Control-Allow-Credentials', 'true');
-
-  if (request.method === 'OPTIONS') {
-    logger.info('Hook returning 204 No Content for CORS preflight request.');
-    return void reply.status(204).send();
-  }
-
-  done();
-};
-
-export const allowAllOriginsHookHandlerTest = (
+export const allowAllOriginsHookHandler = (
   request: FastifyRequest,
   reply: FastifyReply,
 ) => {

packages/server/api/src/app/user-info/user-info.module.ts

@@ -1,10 +1,7 @@
 import { FastifyPluginAsyncTypebox } from '@fastify/type-provider-typebox';
 import { AppSystemProp, logger, system } from '@openops/server-shared';
 import { ALL_PRINCIPAL_TYPES } from '@openops/shared';
-import {
-  allowAllOriginsHookHandler,
-  allowAllOriginsHookHandlerTest,
-} from '../helper/allow-all-origins-hook-handler';
+import { allowAllOriginsHookHandler } from '../helper/allow-all-origins-hook-handler';
 import { getVerifiedUser } from './cloud-auth';
 
 export const userInfoModule: FastifyPluginAsyncTypebox = async (app) => {
@@ -25,18 +22,16 @@ export const userInfoController: FastifyPluginAsyncTypebox = async (app) => {
   }
 
   // user-info is available on any origin
-  // app.addHook('onRequest', allowAllOriginsHookHandler);
-
   app.options(
-    '/*',
+    '*',
     {
       config: {
         allowedPrincipals: ALL_PRINCIPAL_TYPES,
         skipAuth: true,
         cors: false,
       },
     },
-    allowAllOriginsHookHandlerTest,
+    allowAllOriginsHookHandler,
   );
 
   app.get(