slskd: EXTENSIONS=mp3 is downloading flac and aif files, not mp3s

[maroonedbaboon]

Thanks for the app!

I have Explo hooked to slskd and downloading files.
However I want to limit slskd to only download 320kbps mp3s.

So I used
EXTENSIONS=mp3
MIN_BITRATE=320

but I'm getting flacs and aifs, no mp3s at all.

Here's my Explo config in the UI

TZ=UTC
DAILY_JAMS_SCHEDULE=15 01 * * *
LISTENBRAINZ_USER=Blah
LISTENBRAINZ_DISCOVERY=playlist
WEEKLY_EXPLORATION_FLAGS=--playlist weekly-exploration --clean-downloads
WEEKLY_JAMS_FLAGS=--playlist weekly-jams --clean-downloads
DAILY_JAMS_FLAGS=--playlist daily-jams --clean-downloads
WEEKLY_EXPLORATION_SCHEDULE=15 00 * * 2
WEEKLY_JAMS_SCHEDULE=30 00 * * 1
EXPLO_SYSTEM=plex
API_KEY=Blak
LIBRARY_NAME=Music
SYSTEM_URL=http://192.168.0.119:32400
WIZARD_COMPLETE=true
DOWNLOAD_SERVICES=slskd
USE_SUBDIRECTORY=true
SLSKD_URL=http://192.168.0.119:5030
SLSKD_API_KEY=Blah
EXTENSIONS=mp3
MIN_BITRATE=320
MIGRATE_DOWNLOADS=false
UI_USERNAME=Explo
UI_PASSWORD=Blah
LOG_LEVEL=DEBUG

and my Explo compose.yaml to confirm nothing is interfering from there.

services:
explo:
image: ghcr.io/lumepart/explo:latest
restart: unless-stopped
container_name: explo
ports:
- "7288:7288"
environment:
- PUID=99
- PGID=100
- UMASK=002
volumes:
- /mnt/user/appdata/explo/.env:/opt/explo/.env
- /mnt/user/appdata/explo/config:/opt/explo/config
- /mnt/user/appdata/explo/data:/data
- /mnt/ssd/overflow/music/:/slskd/ # <-- ADD THIS LINE (Point to your actual slskd completed downloads path)

Looking at slskd, in the UI and logs, I can't see any sign of "searching for blah with parameters mp3 / 320"
Looking at explo logs, I can't see anything about "searching with..."
Searching manually, I see 320kbps mp3 files available.

Pardon if this is on me, but I can't see it.

Thank you!

[maroonedbaboon]

Got this working. Here are the changes made to the slskd.go file

If or, not if and
Old:

• if !slices.Contains(c.Cfg.Filters.Extensions, file.Extension) && ContainsKeyword(...) {

New:

• if !slices.Contains(c.Cfg.Filters.Extensions, file.Extension) || ContainsKeyword(...) {

If you type MINBITRATE=320, it wants 321

• if file.BitRate > 0 && file.BitRate <= c.Cfg.Filters.MinBitRate {

New:

• if file.BitRate > 0 && file.BitRate < c.Cfg.Filters.MinBitRate {

Same deal with the MIN_BIT_DEPTH

slskd is returning "mp3" for everything - flac, aif
So don't trust and rely on the actual extension.

Old:
file.Extension = strings.TrimPrefix(strings.ToLower(file.Extension), ".")
if file.Extension == "" {
extension := strings.TrimPrefix(strings.ToLower(filepath.Ext(string(file.Name))), ".")
file.Extension = util.AlnumOnly(extension) // sanitize extension incase of bad chars
}

New:
nameExtension := strings.TrimPrefix(strings.ToLower(filepath.Ext(string(file.Name))), ".")
nameExtension = util.AlnumOnly(nameExtension)

reportedExtension := strings.TrimPrefix(strings.ToLower(file.Extension), ".")

if nameExtension != "" {
if reportedExtension != "" && reportedExtension != nameExtension {
slog.Debug("extension mismatch between filename and metadata, using filename",
"track", trackLabel, "filename", file.Name, "reportedExtension", reportedExtension, "actualExtension", nameExtension)
}
file.Extension = nameExtension
} else {
file.Extension = reportedExtension
}

[LumePart]

Thanks! will get it fixed in the next version.