yadunut/clan-core
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

cli: fix unnecessary sudo when unspecified user defaults to root

Browse Source
This commit is contained in:
Michael Hoang
2025-04-22 17:43:15 +10:00
parent b5901d5577
commit 91a2739ea3
2 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
pkgs/clan-cli/clan_cli/ssh/host.py
View File

@@ -38,13 +38,15 @@ class Host:
def __post_init__(self) -> None: def __post_init__(self) -> None:
if not self.command_prefix: if not self.command_prefix:
self.command_prefix = self.host self.command_prefix = self.host
if not self.user:
self.user = "root"
def __str__(self) -> str: def __str__(self) -> str:
return self.target return self.target
@property @property
def target(self) -> str: def target(self) -> str:
return f"{self.user or 'root'}@{self.host}" return f"{self.user}@{self.host}"
@classmethod @classmethod
def from_host(cls, host: "Host") -> "Host": def from_host(cls, host: "Host") -> "Host":

4
pkgs/clan-cli/clan_cli/tests/test_ssh_remote.py
View File

@@ -114,7 +114,9 @@ def test_parse_deployment_address(
assert result.host == expected_host assert result.host == expected_host
assert result.port == expected_port assert result.port == expected_port
assert result.user == expected_user assert result.user == expected_user or (
expected_user == "" and result.user == "root"
)
assert result.ssh_options == expected_options assert result.ssh_options == expected_options