mirrors/hedgedoc
1
0
Fork 0
mirror of https://github.com/hedgedoc/hedgedoc.git synced 2025-05-13 22:54:42 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix: change logging from id to publicId in media upload

Browse source 
Signed-off-by: Yannick Bungers <git@innay.de>
This commit is contained in:
Yannick Bungers 2023-04-23 21:57:07 +02:00 • committed by Yannick Bungers
parent c20e20b30a
commit dad60a25ea
2 changed files with 8 additions and 15 deletions
Download patch file Download diff file Expand all files Collapse all files

6
backend/src/api/private/media/media.controller.ts
View file

@ -88,12 +88,12 @@ export class MediaController {
} }
if (user) { if (user) {
this.logger.debug( this.logger.debug(
`Received filename '${file.originalname}' for note '${note.id}' from user '${user.username}'`, `Received filename '${file.originalname}' for note '${note.publicId}' from user '${user.username}'`,
'uploadMedia', 'uploadMedia',
); );
} else { } else {
this.logger.debug( this.logger.debug(
`Received filename '${file.originalname}' for note '${note.id}' from not logged in user`, `Received filename '${file.originalname}' for note '${note.publicId}' from not logged in user`,
'uploadMedia', 'uploadMedia',
); );
} }
@ -127,7 +127,7 @@ export class MediaController {
const mediaUploadNote = await mediaUpload.note; const mediaUploadNote = await mediaUpload.note;
throw new PermissionError( throw new PermissionError(
`Neither file '${filename}' nor note '${ `Neither file '${filename}' nor note '${
mediaUploadNote?.id ?? 'unknown' mediaUploadNote?.publicId ?? 'unknown'
}'is owned by '${user.username}'`, }'is owned by '${user.username}'`,
); );
} }

17
backend/src/api/public/media/media.controller.ts
View file

@ -93,17 +93,10 @@ export class MediaController {
if (file === undefined) { if (file === undefined) {
throw new BadRequestException('Request does not contain a file'); throw new BadRequestException('Request does not contain a file');
} }
if (user) { this.logger.debug(
this.logger.debug( `Received filename '${file.originalname}' for note '${note.publicId}' from user '${user.username}'`,
`Received filename '${file.originalname}' for note '${note.publicId}' from user '${user.username}'`, 'uploadMedia',
'uploadMedia', );
);
} else {
this.logger.debug(
`Received filename '${file.originalname}' for note '${note.publicId}' from not logged in user`,
'uploadMedia',
);
}
const upload = await this.mediaService.saveFile(file.buffer, user, note); const upload = await this.mediaService.saveFile(file.buffer, user, note);
return await this.mediaService.toMediaUploadDto(upload); return await this.mediaService.toMediaUploadDto(upload);
} }
@ -134,7 +127,7 @@ export class MediaController {
const mediaUploadNote = await mediaUpload.note; const mediaUploadNote = await mediaUpload.note;
throw new PermissionError( throw new PermissionError(
`Neither file '${filename}' nor note '${ `Neither file '${filename}' nor note '${
mediaUploadNote?.id ?? 'unknown' mediaUploadNote?.publicId ?? 'unknown'
}'is owned by '${user.username}'`, }'is owned by '${user.username}'`,
); );
} }