Remove debug logs

safe-global · Jan 16, 2025 · c2c24fb · c2c24fb
1 parent 1e51f9e
commit c2c24fb
Show file tree

Hide file tree

Showing 6 changed files with 0 additions and 57 deletions.
diff --git a/migrations/1733761866546-cleanNotificationData.ts b/migrations/1733761866546-cleanNotificationData.ts
diff --git a/migrations/1734992798452-cleanNotificationData.ts b/migrations/1734992798452-cleanNotificationData.ts
diff --git a/src/domain/hooks/helpers/event-notifications.helper.ts b/src/domain/hooks/helpers/event-notifications.helper.ts
@@ -68,22 +68,18 @@ export class EventNotificationsHelper {
    * @param event - {@link Event} to notify about
    */
   public async onEventEnqueueNotifications(event: Event): Promise<unknown> {
-    console.log('NotificationToEnqueue:', JSON.stringify(event));
     if (!this.isEventToNotify(event)) {
       return;
     }
 
     const subscriptions = await this.getRelevantSubscribers(event);
 
-    console.log('NotificationSubscriptions:', JSON.stringify(subscriptions));
-
     return await Promise.allSettled(
       subscriptions.map(async (subscription) => {
         const data = await this.mapEventNotification(
           event,
           subscription.subscriber,
         );
-        console.log('NotificationData:', JSON.stringify(data));
 
         if (!data) {
           return;

diff --git a/src/domain/hooks/hooks.repository.ts b/src/domain/hooks/hooks.repository.ts
@@ -47,11 +47,6 @@ export class HooksRepositoryWithNotifications
   }
 
   async onEvent(event: Event): Promise<unknown> {
-    console.log(
-      'Incoming Event:',
-      JSON.stringify(event),
-      '-----------------------------',
-    );
     const isSupportedChainId = await this.eventCacheHelper.isSupportedChainMemo(
       event.chainId,
     );

diff --git a/src/routes/notifications/v1/notifications.controller.ts b/src/routes/notifications/v1/notifications.controller.ts
@@ -67,8 +67,6 @@ export class NotificationsController {
       return await this.notificationsService.registerDevice(registerDeviceDto);
     }
 
-    console.log('PushNotificationRequest:', JSON.stringify(registerDeviceDto));
-
     if (registerDeviceDto.timestamp) {
       this.validateTimestamp(parseInt(registerDeviceDto.timestamp));
     }
@@ -77,8 +75,6 @@ export class NotificationsController {
     const compatibleV2Requests =
       await this.createV2RegisterDto(registerDeviceDto);
 
-    console.log('CompatibleV2Requests:', JSON.stringify(compatibleV2Requests));
-
     const v2Requests = [];
 
     const deleteAllDeviceOwners =
@@ -88,7 +84,6 @@ export class NotificationsController {
       // Some clients, such as the mobile app, do not call the delete endpoint to remove an owner key.
       // Instead, they resend the updated list of owners without the key they want to delete.
       // In such cases, we need to clear all the previous owners to ensure the update is applied correctly.
-      console.log('deleteDeviceOwnersFor:', registerDeviceDto.uuid);
       await this.notificationServiceV2.deleteDeviceOwners(
         registerDeviceDto.uuid,
       );
@@ -112,10 +107,6 @@ export class NotificationsController {
     if (registerDeviceDto.uuid) {
       const unregistrationRequests = [];
       for (const safeRegistration of registerDeviceDto.safeRegistrations) {
-        console.log(
-          'TokensToRemoveFromTheOldTxService:',
-          JSON.stringify(registerDeviceDto.uuid),
-        );
         unregistrationRequests.push(
           this.notificationsService.unregisterDevice({
             chainId: safeRegistration.chainId,
@@ -132,11 +123,6 @@ export class NotificationsController {
               'code' in result.reason &&
               result.reason.code !== 404
             ) {
-              console.log(
-                'ErrorOccuredUnregisteringTheDecive:',
-                result.reason,
-                result.reason.code,
-              );
               this.loggingService.error(result.reason);
             }
           }
@@ -276,7 +262,6 @@ export class NotificationsController {
     @Param('chainId') chainId: string,
     @Param('uuid', new ValidationPipe(UuidSchema)) uuid: UUID,
   ): Promise<void> {
-    console.log('ControllerUnregisterDevice:', chainId, uuid);
     if (this.isPushNotificationV2Enabled) {
       return await this.unregisterDeviceV2Compatible(chainId, uuid);
     }
@@ -291,9 +276,7 @@ export class NotificationsController {
     try {
       await this.notificationServiceV2.deleteDevice(uuid);
     } catch (error: unknown) {
-      console.log('NotificationUnregisterErrorNotFound:', error);
       if (error instanceof NotFoundException) {
-        console.log('NotificationUnregisterErrorNotFound:', error);
         // Do not throw a NotFound error when attempting to remove the token from the CGW,
         // This ensures the TX service remove method is called
       } else {
@@ -302,10 +285,8 @@ export class NotificationsController {
     }
 
     try {
-      console.log('here-----');
       await this.notificationsService.unregisterDevice({ chainId, uuid });
     } catch (error: unknown) {
-      console.log('NotificationUnregisterErrorTX---:', error);
       // The token might already have been removed from the TX service.
       // If this happens, the TX service will throw a 404 error, but it is safe to ignore it.
       const errorObject = error as { code?: number };
@@ -322,7 +303,6 @@ export class NotificationsController {
     @Param('safeAddress', new ValidationPipe(AddressSchema))
     safeAddress: `0x${string}`,
   ): Promise<void> {
-    console.log('ControllerUnregisterSafe:', chainId, uuid, safeAddress);
     if (this.isPushNotificationV2Enabled) {
       return this.unregisterSafeV2Compatible(chainId, uuid, safeAddress);
     }
@@ -362,7 +342,6 @@ export class NotificationsController {
         safeAddress,
       });
     } catch (error: unknown) {
-      console.log('NotificationUnregisterError:', error);
       // The token might already have been removed from the TX service.
       // If this happens, the TX service will throw a 404 error, but it is safe to ignore it.
       const errorObject = error as { code?: number };

diff --git a/src/routes/notifications/v1/notifications.service.ts b/src/routes/notifications/v1/notifications.service.ts
@@ -84,7 +84,6 @@ export class NotificationsService {
     chainId: string;
     uuid: string;
   }): Promise<void> {
-    console.log('UnregisterDevice:', args.uuid, args.chainId);
     return this.notificationsRepository.unregisterDevice(args);
   }