Skip to content

Commit

Permalink
fix: Terminal output from plugin installation is now safely decoded (m…
Browse files Browse the repository at this point in the history
  • Loading branch information
edgarrmondragon committed Mar 6, 2024
1 parent 763994e commit c4b7eeb
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/meltano/core/venv_service.py
Expand Up @@ -325,6 +325,7 @@ async def create(self) -> Process:
async def extract_stderr(proc: Process):
return (await t.cast(asyncio.StreamReader, proc.stdout).read()).decode(
"unicode_escape",
errors="replace",
)

try:
Expand Down Expand Up @@ -441,7 +442,7 @@ async def extract_stderr(proc: Process) -> str | None: # pragma: no cover
if not proc.stdout:
return None

return (await proc.stdout.read()).decode("unicode_escape")
return (await proc.stdout.read()).decode("unicode_escape", errors="replace")

try:
return await exec_async(
Expand Down

0 comments on commit c4b7eeb

Please sign in to comment.