Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion litellm/litellm_core_utils/audio_utils/utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ def process_audio_file(audio_file: FileTypes) -> ProcessedAudioFile:
file_path = str(audio_file)
with open(file_path, "rb") as f:
file_content = f.read()
filename = file_path.split("/")[-1]
filename = os.path.basename(file_path)
elif isinstance(audio_file, tuple):
# Tuple format: (filename, content, content_type) or (filename, content)
if len(audio_file) >= 2:
Expand Down
29 changes: 29 additions & 0 deletions tests/test_litellm/litellm_core_utils/test_audio_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,35 @@ def test_process_bare_str_path_rejected(self):
with pytest.raises(ValueError, match="does not accept bare str inputs"):
process_audio_file("/etc/passwd")

def test_process_windows_pathlib_input_basename(self):
"""A Windows path must yield only the basename, not the full drive path.

On Windows, str(Path('C:/dir/name.mp3')) is 'C:\\dir\\name.mp3', which
contains no '/'. Deriving the filename with str.split('/')[-1] would send
the entire path (drive letter and backslashes) to the provider as the
multipart file name. os.path is patched to ntpath here to reproduce
Windows semantics on any host.
"""
import ntpath
from pathlib import PureWindowsPath

windows_path = PureWindowsPath("C:/Users/me/audio recording.mp3")
test_content = b"test audio content"

with (
patch("litellm.litellm_core_utils.audio_utils.utils.os.path", ntpath),
patch(
"litellm.litellm_core_utils.audio_utils.utils.open",
mock_open(read_data=test_content),
create=True,
),
):
result = process_audio_file(windows_path)

assert result.file_content == test_content
assert result.filename == "audio recording.mp3"
assert result.content_type == "audio/mpeg"

def test_process_tuple_input_with_bytes(self):
"""Test processing tuple input with bytes content"""
filename = "test.wav"
Expand Down
Loading