♻️ remove duplicate onConnected route

Author: vogelskamp

src/index.ts

@@ -7,7 +7,6 @@ import { TokenCacheStorage } from './cache';
 import { errorHandlerMiddleware, extractHeaderMiddleware } from './middlewares';
 import {
   Adapter,
-  BridgeRequest,
   BridgeRequestWithTimestamp,
   ContactCache,
   Controller,
@@ -138,10 +137,6 @@ export function start(
     controller.handleWebhook(req, res, next),
   );
 
-  app.post('/on-connected', (req: BridgeRequest<unknown>, res, next) =>
-    controller.handleConnectedEvent(req, res, next),
-  );
-
   app.use(errorHandlerMiddleware);
 
   customRouters.forEach(({ path, router }) => app.use(path, router));

src/models/adapter.model.ts

@@ -88,7 +88,6 @@ export interface Adapter {
     req: Request,
     res?: Response,
   ) => Promise<{ apiKey: string; apiUrl: string }>;
-  onConnected?: (config: Config) => Promise<void>;
   handleWebhook?: (req: Request) => Promise<IntegrationsEvent[]>;
   verifyWebhookRequest?: (req: Request) => Promise<boolean>;
 }

src/models/controller.model.ts

@@ -1462,37 +1462,4 @@ export class Controller {
       next(error);
     }
   }
-
-  public async onConnected(
-    req: BridgeRequest<unknown>,
-    res: Response,
-    next: NextFunction,
-  ) {
-    const { providerConfig } = req;
-
-    try {
-      if (!providerConfig) {
-        throw new ServerError(400, 'Missing parameters');
-      }
-
-      if (!this.adapter.onConnected) {
-        infoLogger(
-          'onConnected',
-          'No onConnected handler defined',
-          providerConfig.apiKey,
-        );
-        return;
-      }
-
-      await this.adapter.onConnected(providerConfig);
-    } catch (error) {
-      errorLogger(
-        'onConnected',
-        'Could not handle onConnected event',
-        '',
-        error || 'Unknown',
-      );
-      next(error);
-    }
-  }
 }