Skip to content

Add detection of files without audio tracks (requireaudio option)#130

Merged
aetaric merged 2 commits intoaetaric:mainfrom
TOomaAh:feature/has-audio-check
Apr 4, 2025
Merged

Add detection of files without audio tracks (requireaudio option)#130
aetaric merged 2 commits intoaetaric:mainfrom
TOomaAh:feature/has-audio-check

Conversation

@TOomaAh
Copy link
Copy Markdown
Contributor

@TOomaAh TOomaAh commented Apr 2, 2025

Add detection of files without audio tracks

Description

This PR adds a new feature allowing Checkrr to detect video files that don't contain any audio track. When such a file is detected, it's flagged as problematic and processed according to the user's configuration (deletion or reacquisition via Sonarr/Radarr/Lidarr).

New features

  • Added configuration option requireaudio: true|false to enable/disable this check
  • Implemented detection logic for missing audio tracks in the checkFile() function
  • Added appropriate log messages to trace these detections

How to test

  1. Add requireaudio: true to your configuration file
  2. Run Checkrr on a library containing video files without audio tracks
  3. Check logs and notifications to confirm these files are properly detected

Notes

Files without audio tracks are reported with reason "no audio streams" in the database and logs.

@aetaric aetaric self-requested a review April 4, 2025 05:22
@aetaric aetaric self-assigned this Apr 4, 2025
@aetaric
Copy link
Copy Markdown
Owner

aetaric commented Apr 4, 2025

@TOomaAh This looks good code-wise, can you add the option to the example config as well so I can merge this and not forget to do it later?

@TOomaAh
Copy link
Copy Markdown
Contributor Author

TOomaAh commented Apr 4, 2025

@TOomaAh This looks good code-wise, can you add the option to the example config as well so I can merge this and not forget to do it later?

Done

@aetaric aetaric merged commit f530b5e into aetaric:main Apr 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants