Skip to content

Commit 3df2824

Browse files
committed
Use octal for default permissions
1 parent 96ef6c5 commit 3df2824

File tree

2 files changed

+11
-2
lines changed

2 files changed

+11
-2
lines changed

agentpluginapi/i_linux_agent_command_builder.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ class LinuxDownloadOptions(InfectionMonkeyBaseModel):
2222

2323
class LinuxSetPermissionsOptions(InfectionMonkeyBaseModel):
2424
file_path: PurePosixPath
25-
permissions: int = Field(ge=0, le=0o777, default=700)
25+
permissions: int = Field(ge=0, le=0o777, default=0o700)
2626

2727

2828
class LinuxRunOptions(InfectionMonkeyBaseModel):

tests/test_i_linux_agent_command_builder.py

+10-1
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
import pytest
44

5-
from agentpluginapi import DropperExecutionMode, LinuxRunOptions
5+
from agentpluginapi import DropperExecutionMode, LinuxRunOptions, LinuxSetPermissionsOptions
66

77

88
@pytest.mark.parametrize(
@@ -15,3 +15,12 @@ def test_linux_run_options(dropper_execution_mode: DropperExecutionMode):
1515
dropper_execution_mode=dropper_execution_mode,
1616
dropper_destination_path=PurePosixPath("/tmp/dropper"),
1717
)
18+
19+
20+
@pytest.mark.parametrize("permissions", [0o1000, -0o1])
21+
def test_linux_permissions_options(permissions: int):
22+
with pytest.raises(ValueError):
23+
LinuxSetPermissionsOptions(
24+
agent_destination_path=PurePosixPath("/tmp/agent"),
25+
permissions=permissions,
26+
)

0 commit comments

Comments
 (0)