chore: show error logs properly

sampaiodiego · sampaiodiego · commit aa7354ebae18 · 2025-12-04T17:26:02.000-03:00
diff --git a/ee/packages/federation-matrix/src/FederationMatrix.ts b/ee/packages/federation-matrix/src/FederationMatrix.ts
@@ -228,7 +228,7 @@ export class FederationMatrix extends ServiceClass implements IFederationMatrixS
 
 			return matrixRoomResult;
 		} catch (error) {
-			this.logger.error('Failed to create room:', error);
+			this.logger.error(error, 'Failed to create room');
 			throw error;
 		}
 	}
@@ -302,7 +302,7 @@ export class FederationMatrix extends ServiceClass implements IFederationMatrixS
 							userIdSchema.parse(actualMatrixUserId),
 						);
 					} catch (error) {
-						this.logger.error('Error creating or updating bridged user for DM:', error);
+						this.logger.error(error, 'Error creating or updating bridged user for DM');
 					}
 				}
 			}
@@ -311,9 +311,9 @@ export class FederationMatrix extends ServiceClass implements IFederationMatrixS
 				mrid: matrixRoomResult.room_id,
 				origin: this.serverName,
 			});
-			this.logger.debug('Direct message room creation completed successfully', room._id);
+			this.logger.debug(room._id, 'Direct message room creation completed successfully');
 		} catch (error) {
-			this.logger.error('Failed to create direct message room:', error);
+			this.logger.error(error, 'Failed to create direct message room');
 			throw error;
 		}
 	}
@@ -367,9 +367,10 @@ export class FederationMatrix extends ServiceClass implements IFederationMatrixS
 
 			return lastEventId;
 		} catch (error) {
-			this.logger.error('Failed to handle file message', {
+			this.logger.error({
+				msg: 'Failed to handle file message',
 				messageId: message._id,
-				error,
+				err: error,
 			});
 			throw error;
 		}
@@ -464,7 +465,7 @@ export class FederationMatrix extends ServiceClass implements IFederationMatrixS
 
 			this.logger.debug('Message sent to Matrix successfully:', result.eventId);
 		} catch (error) {
-			this.logger.error('Failed to send message to Matrix:', error);
+			this.logger.error(error, 'Failed to send message to Matrix');
 			throw error;
 		}
 	}
@@ -526,7 +527,7 @@ export class FederationMatrix extends ServiceClass implements IFederationMatrixS
 
 			this.logger.debug('Message Redaction sent to Matrix successfully:', eventId);
 		} catch (error) {
-			this.logger.error('Failed to send redaction to Matrix:', error);
+			this.logger.error(error, 'Failed to send redaction to Matrix');
 			throw error;
 		}
 	}
@@ -560,7 +561,7 @@ export class FederationMatrix extends ServiceClass implements IFederationMatrixS
 				}),
 			);
 		} catch (error) {
-			this.logger.error('Failed to invite a user to Matrix:', error);
+			this.logger.error(error, 'Failed to invite a user to Matrix');
 			throw error;
 		}
 	}
@@ -597,7 +598,7 @@ export class FederationMatrix extends ServiceClass implements IFederationMatrixS
 
 			this.logger.debug('Reaction sent to Matrix successfully:', eventId);
 		} catch (error) {
-			this.logger.error('Failed to send reaction to Matrix:', error);
+			this.logger.error(error, 'Failed to send reaction to Matrix');
 			throw error;
 		}
 	}
@@ -651,7 +652,7 @@ export class FederationMatrix extends ServiceClass implements IFederationMatrixS
 				break;
 			}
 		} catch (error) {
-			this.logger.error('Failed to remove reaction from Matrix:', error);
+			this.logger.error(error, 'Failed to remove reaction from Matrix');
 			throw error;
 		}
 	}
@@ -679,7 +680,7 @@ export class FederationMatrix extends ServiceClass implements IFederationMatrixS
 
 			this.logger.info(`User ${user.username} left Matrix room ${room.federation.mrid} successfully`);
 		} catch (error) {
-			this.logger.error('Failed to leave room in Matrix:', error);
+			this.logger.error(error, 'Failed to leave room in Matrix');
 			throw error;
 		}
 	}
@@ -703,7 +704,7 @@ export class FederationMatrix extends ServiceClass implements IFederationMatrixS
 
 			this.logger.info(`User ${removedUser.username} was kicked from Matrix room ${room.federation.mrid} by ${userWhoRemoved.username}`);
 		} catch (error) {
-			this.logger.error('Failed to kick user from Matrix room:', error);
+			this.logger.error(error, 'Failed to kick user from Matrix room');
 			throw error;
 		}
 	}
@@ -738,7 +739,7 @@ export class FederationMatrix extends ServiceClass implements IFederationMatrixS
 
 			this.logger.debug('Message updated in Matrix successfully:', eventId);
 		} catch (error) {
-			this.logger.error('Failed to update message in Matrix:', error);
+			this.logger.error(error, 'Failed to update message in Matrix');
 			throw error;
 		}
 	}
diff --git a/ee/packages/federation-matrix/src/events/edu.ts b/ee/packages/federation-matrix/src/events/edu.ts
@@ -27,7 +27,7 @@ export const edus = async (emitter: Emitter<HomeserverEventSignatures>) => {
 				roomId: matrixRoom._id,
 			});
 		} catch (error) {
-			logger.error('Error handling Matrix typing event:', error);
+			logger.error(error, 'Error handling Matrix typing event');
 		}
 	});
 
@@ -79,7 +79,7 @@ export const edus = async (emitter: Emitter<HomeserverEventSignatures>) => {
 			});
 			logger.debug(`Updated presence for user ${matrixUser._id} to ${status} from Matrix federation`);
 		} catch (error) {
-			logger.error('Error handling Matrix presence event:', error);
+			logger.error(error, 'Error handling Matrix presence event');
 		}
 	});
 };
diff --git a/ee/packages/federation-matrix/src/events/member.ts b/ee/packages/federation-matrix/src/events/member.ts
@@ -231,7 +231,7 @@ export function member(emitter: Emitter<HomeserverEventSignatures>) {
 					logger.warn(`Unknown membership type: ${event.content.membership}`);
 			}
 		} catch (error) {
-			logger.error('Failed to process Matrix membership event:', error);
+			logger.error(error, 'Failed to process Matrix membership event');
 		}
 	});
 }
diff --git a/ee/packages/federation-matrix/src/events/message.ts b/ee/packages/federation-matrix/src/events/message.ts
@@ -413,7 +413,7 @@ export function message(emitter: Emitter<HomeserverEventSignatures>) {
 
 			await Message.deleteMessage(user, rcMessage);
 		} catch (error) {
-			logger.error('Failed to process Matrix removal redaction:', error);
+			logger.error(error, 'Failed to process Matrix removal redaction');
 		}
 	});
 }
diff --git a/ee/packages/federation-matrix/src/events/reaction.ts b/ee/packages/federation-matrix/src/events/reaction.ts
@@ -43,7 +43,7 @@ export function reaction(emitter: Emitter<HomeserverEventSignatures>) {
 			await Message.reactToMessage(user._id, reactionEmoji, rcMessage._id, true);
 			await Messages.setFederationReactionEventId(internalUsername, rcMessage._id, reactionEmoji, eventId);
 		} catch (error) {
-			logger.error('Failed to process Matrix reaction:', error);
+			logger.error(error, 'Failed to process Matrix reaction');
 		}
 	});
 
@@ -87,7 +87,7 @@ export function reaction(emitter: Emitter<HomeserverEventSignatures>) {
 			await Message.reactToMessage(user._id, reactionEmoji, rcMessage._id, false);
 			await Messages.unsetFederationReactionEventId(redactedEventId, rcMessage._id, reactionEmoji);
 		} catch (error) {
-			logger.error('Failed to process Matrix reaction redaction:', error);
+			logger.error(error, 'Failed to process Matrix reaction redaction');
 		}
 	});
 }
diff --git a/ee/packages/federation-matrix/src/services/MatrixMediaService.ts b/ee/packages/federation-matrix/src/services/MatrixMediaService.ts
@@ -55,7 +55,7 @@ export class MatrixMediaService {
 
 			return mxcUri;
 		} catch (error) {
-			logger.error('Error preparing file for Matrix:', error);
+			logger.error(error, 'Error preparing file for Matrix');
 			throw error;
 		}
 	}
@@ -74,7 +74,7 @@ export class MatrixMediaService {
 
 			return file;
 		} catch (error) {
-			logger.error('Error retrieving local file:', error);
+			logger.error(error, 'Error retrieving local file');
 			return null;
 		}
 	}
@@ -130,7 +130,7 @@ export class MatrixMediaService {
 
 			return uploadedFile._id;
 		} catch (error) {
-			logger.error('Error downloading and storing remote file:', error);
+			logger.error(error, 'Error downloading and storing remote file');
 			throw error;
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -228,7 +228,7 @@ export class FederationMatrix extends ServiceClass implements IFederationMatrixS`
`228`	`228`
`229`	`229`	`return matrixRoomResult;`
`230`	`230`	`} catch (error) {`
`231`		`- this.logger.error('Failed to create room:', error);`
	`231`	`+ this.logger.error(error, 'Failed to create room');`
`232`	`232`	`throw error;`
`233`	`233`	`}`
`234`	`234`	`}`
`@@ -302,7 +302,7 @@ export class FederationMatrix extends ServiceClass implements IFederationMatrixS`
`302`	`302`	`userIdSchema.parse(actualMatrixUserId),`
`303`	`303`	`);`
`304`	`304`	`} catch (error) {`
`305`		`- this.logger.error('Error creating or updating bridged user for DM:', error);`
	`305`	`+ this.logger.error(error, 'Error creating or updating bridged user for DM');`
`306`	`306`	`}`
`307`	`307`	`}`
`308`	`308`	`}`
`@@ -311,9 +311,9 @@ export class FederationMatrix extends ServiceClass implements IFederationMatrixS`
`311`	`311`	`mrid: matrixRoomResult.room_id,`
`312`	`312`	`origin: this.serverName,`
`313`	`313`	`});`
`314`		`- this.logger.debug('Direct message room creation completed successfully', room._id);`
	`314`	`+ this.logger.debug(room._id, 'Direct message room creation completed successfully');`
`315`	`315`	`} catch (error) {`
`316`		`- this.logger.error('Failed to create direct message room:', error);`
	`316`	`+ this.logger.error(error, 'Failed to create direct message room');`
`317`	`317`	`throw error;`
`318`	`318`	`}`
`319`	`319`	`}`
`@@ -367,9 +367,10 @@ export class FederationMatrix extends ServiceClass implements IFederationMatrixS`
`367`	`367`
`368`	`368`	`return lastEventId;`
`369`	`369`	`} catch (error) {`
`370`		`- this.logger.error('Failed to handle file message', {`
	`370`	`+ this.logger.error({`
	`371`	`+ msg: 'Failed to handle file message',`
`371`	`372`	`messageId: message._id,`
`372`		`- error,`
	`373`	`+ err: error,`
`373`	`374`	`});`
`374`	`375`	`throw error;`
`375`	`376`	`}`
`@@ -464,7 +465,7 @@ export class FederationMatrix extends ServiceClass implements IFederationMatrixS`
`464`	`465`
`465`	`466`	`this.logger.debug('Message sent to Matrix successfully:', result.eventId);`
`466`	`467`	`} catch (error) {`
`467`		`- this.logger.error('Failed to send message to Matrix:', error);`
	`468`	`+ this.logger.error(error, 'Failed to send message to Matrix');`
`468`	`469`	`throw error;`
`469`	`470`	`}`
`470`	`471`	`}`
`@@ -526,7 +527,7 @@ export class FederationMatrix extends ServiceClass implements IFederationMatrixS`
`526`	`527`
`527`	`528`	`this.logger.debug('Message Redaction sent to Matrix successfully:', eventId);`
`528`	`529`	`} catch (error) {`
`529`		`- this.logger.error('Failed to send redaction to Matrix:', error);`
	`530`	`+ this.logger.error(error, 'Failed to send redaction to Matrix');`
`530`	`531`	`throw error;`
`531`	`532`	`}`
`532`	`533`	`}`
`@@ -560,7 +561,7 @@ export class FederationMatrix extends ServiceClass implements IFederationMatrixS`
`560`	`561`	`}),`
`561`	`562`	`);`
`562`	`563`	`} catch (error) {`
`563`		`- this.logger.error('Failed to invite a user to Matrix:', error);`
	`564`	`+ this.logger.error(error, 'Failed to invite a user to Matrix');`
`564`	`565`	`throw error;`
`565`	`566`	`}`
`566`	`567`	`}`
`@@ -597,7 +598,7 @@ export class FederationMatrix extends ServiceClass implements IFederationMatrixS`
`597`	`598`
`598`	`599`	`this.logger.debug('Reaction sent to Matrix successfully:', eventId);`
`599`	`600`	`} catch (error) {`
`600`		`- this.logger.error('Failed to send reaction to Matrix:', error);`
	`601`	`+ this.logger.error(error, 'Failed to send reaction to Matrix');`
`601`	`602`	`throw error;`
`602`	`603`	`}`
`603`	`604`	`}`
`@@ -651,7 +652,7 @@ export class FederationMatrix extends ServiceClass implements IFederationMatrixS`
`651`	`652`	`break;`
`652`	`653`	`}`
`653`	`654`	`} catch (error) {`
`654`		`- this.logger.error('Failed to remove reaction from Matrix:', error);`
	`655`	`+ this.logger.error(error, 'Failed to remove reaction from Matrix');`
`655`	`656`	`throw error;`
`656`	`657`	`}`
`657`	`658`	`}`
`@@ -679,7 +680,7 @@ export class FederationMatrix extends ServiceClass implements IFederationMatrixS`
`679`	`680`
`680`	`681`	this.logger.info(`User ${user.username} left Matrix room ${room.federation.mrid} successfully`);
`681`	`682`	`} catch (error) {`
`682`		`- this.logger.error('Failed to leave room in Matrix:', error);`
	`683`	`+ this.logger.error(error, 'Failed to leave room in Matrix');`
`683`	`684`	`throw error;`
`684`	`685`	`}`
`685`	`686`	`}`
`@@ -703,7 +704,7 @@ export class FederationMatrix extends ServiceClass implements IFederationMatrixS`
`703`	`704`
`704`	`705`	this.logger.info(`User ${removedUser.username} was kicked from Matrix room ${room.federation.mrid} by ${userWhoRemoved.username}`);
`705`	`706`	`} catch (error) {`
`706`		`- this.logger.error('Failed to kick user from Matrix room:', error);`
	`707`	`+ this.logger.error(error, 'Failed to kick user from Matrix room');`
`707`	`708`	`throw error;`
`708`	`709`	`}`
`709`	`710`	`}`
`@@ -738,7 +739,7 @@ export class FederationMatrix extends ServiceClass implements IFederationMatrixS`
`738`	`739`
`739`	`740`	`this.logger.debug('Message updated in Matrix successfully:', eventId);`
`740`	`741`	`} catch (error) {`
`741`		`- this.logger.error('Failed to update message in Matrix:', error);`
	`742`	`+ this.logger.error(error, 'Failed to update message in Matrix');`
`742`	`743`	`throw error;`
`743`	`744`	`}`
`744`	`745`	`}`
Original file line number	Diff line number	Diff line change
`@@ -231,7 +231,7 @@ export function member(emitter: Emitter<HomeserverEventSignatures>) {`
`231`	`231`	logger.warn(`Unknown membership type: ${event.content.membership}`);
`232`	`232`	`}`
`233`	`233`	`} catch (error) {`
`234`		`- logger.error('Failed to process Matrix membership event:', error);`
	`234`	`+ logger.error(error, 'Failed to process Matrix membership event');`
`235`	`235`	`}`
`236`	`236`	`});`
`237`	`237`	`}`
Original file line number	Diff line number	Diff line change
`@@ -413,7 +413,7 @@ export function message(emitter: Emitter<HomeserverEventSignatures>) {`
`413`	`413`
`414`	`414`	`await Message.deleteMessage(user, rcMessage);`
`415`	`415`	`} catch (error) {`
`416`		`- logger.error('Failed to process Matrix removal redaction:', error);`
	`416`	`+ logger.error(error, 'Failed to process Matrix removal redaction');`
`417`	`417`	`}`
`418`	`418`	`});`
`419`	`419`	`}`
Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,7 @@ export function reaction(emitter: Emitter<HomeserverEventSignatures>) {`
`43`	`43`	`await Message.reactToMessage(user._id, reactionEmoji, rcMessage._id, true);`
`44`	`44`	`await Messages.setFederationReactionEventId(internalUsername, rcMessage._id, reactionEmoji, eventId);`
`45`	`45`	`} catch (error) {`
`46`		`- logger.error('Failed to process Matrix reaction:', error);`
	`46`	`+ logger.error(error, 'Failed to process Matrix reaction');`
`47`	`47`	`}`
`48`	`48`	`});`
`49`	`49`
`@@ -87,7 +87,7 @@ export function reaction(emitter: Emitter<HomeserverEventSignatures>) {`
`87`	`87`	`await Message.reactToMessage(user._id, reactionEmoji, rcMessage._id, false);`
`88`	`88`	`await Messages.unsetFederationReactionEventId(redactedEventId, rcMessage._id, reactionEmoji);`
`89`	`89`	`} catch (error) {`
`90`		`- logger.error('Failed to process Matrix reaction redaction:', error);`
	`90`	`+ logger.error(error, 'Failed to process Matrix reaction redaction');`
`91`	`91`	`}`
`92`	`92`	`});`
`93`	`93`	`}`
Original file line number	Diff line number	Diff line change
`@@ -55,7 +55,7 @@ export class MatrixMediaService {`
`55`	`55`
`56`	`56`	`return mxcUri;`
`57`	`57`	`} catch (error) {`
`58`		`- logger.error('Error preparing file for Matrix:', error);`
	`58`	`+ logger.error(error, 'Error preparing file for Matrix');`
`59`	`59`	`throw error;`
`60`	`60`	`}`
`61`	`61`	`}`
`@@ -74,7 +74,7 @@ export class MatrixMediaService {`
`74`	`74`
`75`	`75`	`return file;`
`76`	`76`	`} catch (error) {`
`77`		`- logger.error('Error retrieving local file:', error);`
	`77`	`+ logger.error(error, 'Error retrieving local file');`
`78`	`78`	`return null;`
`79`	`79`	`}`
`80`	`80`	`}`
`@@ -130,7 +130,7 @@ export class MatrixMediaService {`
`130`	`130`
`131`	`131`	`return uploadedFile._id;`
`132`	`132`	`} catch (error) {`
`133`		`- logger.error('Error downloading and storing remote file:', error);`
	`133`	`+ logger.error(error, 'Error downloading and storing remote file');`
`134`	`134`	`throw error;`
`135`	`135`	`}`
`136`	`136`	`}`