diff --git a/CHANGELOG.md b/CHANGELOG.md index 48ffd9d30..1125ed0fc 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -19,6 +19,7 @@ Changelog](https://keepachangelog.com/en/1.0.0/). - "Kill file" option in the config. #1536 - Netstat collector, because network connection information wasn't used anywhere. #1535 - Checkbox to disable/enable sending log to server. #1537 +- Checkbox for self deleting a monkey agent on cleanup. #1537 ### Fixed - A bug in network map page that caused delay of telemetry log loading. #1545 diff --git a/monkey/infection_monkey/config.py b/monkey/infection_monkey/config.py index ac7084bd3..117875bb4 100644 --- a/monkey/infection_monkey/config.py +++ b/monkey/infection_monkey/config.py @@ -101,9 +101,6 @@ class Configuration(object): # sets whether or not the monkey is alive. if false will stop scanning and exploiting alive = True - # sets whether or not to self delete the monkey executable when stopped - self_delete_in_cleanup = False - # string of the mutex name for single instance singleton_mutex_name = "{2384ec59-0df8-4ab9-918c-843740924a28}" diff --git a/monkey/infection_monkey/example.conf b/monkey/infection_monkey/example.conf index aab1a2a5d..1d3e3235c 100644 --- a/monkey/infection_monkey/example.conf +++ b/monkey/infection_monkey/example.conf @@ -57,7 +57,6 @@ "smb_service_name": "InfectionMonkey", "retry_failed_explotation": true, "self_delete_in_cleanup": true, - "serialize_config": false, "singleton_mutex_name": "{2384ec59-0df8-4ab9-918c-843740924a28}", "skip_exploit_if_file_exist": false, "exploit_user_list": [], diff --git a/monkey/infection_monkey/monkey.py b/monkey/infection_monkey/monkey.py index fdb945978..cc9045408 100644 --- a/monkey/infection_monkey/monkey.py +++ b/monkey/infection_monkey/monkey.py @@ -343,7 +343,7 @@ class InfectionMonkey(object): status = ScanStatus.USED if remove_monkey_dir() else ScanStatus.SCANNED T1107Telem(status, get_monkey_dir_path()).send() - if WormConfiguration.self_delete_in_cleanup and -1 == sys.executable.find("python"): + if -1 == sys.executable.find("python"): try: status = None if "win32" == sys.platform: diff --git a/monkey/monkey_island/cc/services/config_schema/internal.py b/monkey/monkey_island/cc/services/config_schema/internal.py index 023944be2..46b2e7ca3 100644 --- a/monkey/monkey_island/cc/services/config_schema/internal.py +++ b/monkey/monkey_island/cc/services/config_schema/internal.py @@ -60,12 +60,6 @@ INTERNAL = { "monkey propagating to " "a high number of machines", }, - "self_delete_in_cleanup": { - "title": "Self delete on cleanup", - "type": "boolean", - "default": True, - "description": "Should the monkey delete its executable when going down", - }, "use_file_logging": { "title": "Use file logging", "type": "boolean", diff --git a/monkey/tests/data_for_tests/monkey_configs/monkey_config_standard.json b/monkey/tests/data_for_tests/monkey_configs/monkey_config_standard.json index a43c54841..e5c350531 100644 --- a/monkey/tests/data_for_tests/monkey_configs/monkey_config_standard.json +++ b/monkey/tests/data_for_tests/monkey_configs/monkey_config_standard.json @@ -52,7 +52,6 @@ "monkey": { "victims_max_find": 100, "victims_max_exploit": 100, - "self_delete_in_cleanup": true, "use_file_logging": true, "serialize_config": false, "alive": true,