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
8 changes: 8 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,13 @@
# Change Log

## Unreleased

- Fix case where shell cannot be found for `add_process`.
- Fix UTC compatibility for Python 3.10.
- Added retrieval of server version in `Sender`.
- Fixed memory issue when downloading large data files.
- Fixed incorrect exception catch for JSON responses.

## [v2.5.9](https://github.com/simvue-io/python-api/releases/tag/v2.5.9) - 2026-06-26

- Fixed bug with error handling in logging handler
Expand Down
4 changes: 2 additions & 2 deletions simvue/executor.py
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ def __call__(self, *, status_code: int, std_out: str, std_err: str) -> None: ...


def get_current_shell() -> str | None:
"""Return the users current shell executable."""
"""Return the users current shell executable if found."""
try:
_process = psutil.Process(os.getppid())
return os.environ.get("SHELL", _process.exe())
Expand Down Expand Up @@ -156,7 +156,7 @@ def std_err(self, process_id: str) -> str | None:
def _kwarg_assembly(kwargs, executable: str | None) -> list[str]:
_arguments: list[str] = []
_shell_is_pwsh: bool = any(
shell in get_current_shell() for shell in ("pwsh", "powershell")
shell in (get_current_shell() or "") for shell in ("pwsh", "powershell")
)
_exec_is_pwsh: bool = executable in {"pwsh", "powershell", None}
_use_pwsh: bool = _shell_is_pwsh and _exec_is_pwsh
Expand Down
Loading