Cutlery/immich
1
0
Fork 1
mirror of https://github.com/immich-app/immich.git synced 2025-11-03 19:17:11 -05:00
Code Issues Packages Projects Releases Wiki Activity

chore(server): remove unneeded debug logging (#2203)

Browse Source 
* chore(server): commented out debug log line for codecs

* chore(server): removed debug log line for codecs as it's not needed

* Prettier run

* Make the log more useful and move it to verbose level
This commit is contained in:
Skyler Mäntysaari 2023-04-09 04:35:54 +03:00 committed by GitHub
parent fb42a736f1
commit d76b3c8f78
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 6 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
server/libs/domain/src/media/media.service.ts
View File

@ -1,4 +1,4 @@
import { AssetType, TranscodePreset } from '@app/infra/entities'; import { AssetEntity, AssetType, TranscodePreset } from '@app/infra/entities';
import { Inject, Injectable, Logger } from '@nestjs/common'; import { Inject, Injectable, Logger } from '@nestjs/common';
import { join } from 'path'; import { join } from 'path';
import { IAssetRepository, mapAsset, WithoutProperty } from '../asset'; import { IAssetRepository, mapAsset, WithoutProperty } from '../asset';
@ -137,7 +137,7 @@ export class MediaService {
const { ffmpeg: config } = await this.configCore.getConfig(); const { ffmpeg: config } = await this.configCore.getConfig();
const required = this.isTranscodeRequired(mainVideoStream, mainAudioStream, containerExtension, config); const required = this.isTranscodeRequired(asset, mainVideoStream, mainAudioStream, containerExtension, config);
if (!required) { if (!required) {
return; return;
} }
@ -164,6 +164,7 @@ export class MediaService {
} }
private isTranscodeRequired( private isTranscodeRequired(
asset: AssetEntity,
videoStream: VideoStreamInfo, videoStream: VideoStreamInfo,
audioStream: AudioStreamInfo, audioStream: AudioStreamInfo,
containerExtension: string, containerExtension: string,
@ -178,7 +179,9 @@ export class MediaService {
const isTargetAudioCodec = audioStream.codecName === ffmpegConfig.targetAudioCodec; const isTargetAudioCodec = audioStream.codecName === ffmpegConfig.targetAudioCodec;
const isTargetContainer = ['mov,mp4,m4a,3gp,3g2,mj2', 'mp4', 'mov'].includes(containerExtension); const isTargetContainer = ['mov,mp4,m4a,3gp,3g2,mj2', 'mp4', 'mov'].includes(containerExtension);
this.logger.debug(audioStream.codecName, audioStream.codecType, containerExtension); this.logger.verbose(
`${asset.id}: AudioCodecName ${audioStream.codecName}, AudioStreamCodecType ${audioStream.codecType}, containerExtension ${containerExtension}`,
);
const allTargetsMatching = isTargetVideoCodec && isTargetAudioCodec && isTargetContainer; const allTargetsMatching = isTargetVideoCodec && isTargetAudioCodec && isTargetContainer;