Island: Remove PATCH method from api/agent

* Remove config_error from monkey document

monkey/monkey_island/cc/models/monkey.py

@@ -51,7 +51,6 @@ class Monkey(Document):
     # (even with required=False of null=True).
     # See relevant issue: https://github.com/MongoEngine/mongoengine/issues/1904
     parent = ListField(ListField(DynamicField()))
-    config_error = BooleanField()
     critical_services = ListField(StringField())
     pba_results = ListField()
     ttl_ref = ReferenceField(MonkeyTtl)

monkey/monkey_island/cc/resources/monkey.py

@@ -27,29 +27,6 @@ class Monkey(AbstractResource):
     def get(self):
         return {"config": ConfigService.format_flat_config_for_agent()}
 
-    # Used by monkey. can't secure.
-    @TestTelemStore.store_exported_telem
-    def patch(self, guid):
-
-        # TODO: This endpoint appears to be doing 3 things, although only one of them is used
-        # (config_error). The WormConfiguration will be removed in #1960. We should consider
-        # removing this endpoint
-        monkey_json = json.loads(request.data)
-        update = {"$set": {"modifytime": datetime.now()}}
-        monkey = NodeService.get_monkey_by_guid(guid)
-        if "config_error" in monkey_json:
-            update["$set"]["config_error"] = monkey_json["config_error"]
-
-        if "tunnel" in monkey_json:
-            tunnel_host_ip = monkey_json["tunnel"].split(":")[-2].replace("//", "")
-            NodeService.set_monkey_tunnel(monkey["_id"], tunnel_host_ip)
-
-        ttl = create_monkey_ttl_document(DEFAULT_MONKEY_TTL_EXPIRY_DURATION_IN_SECONDS)
-        update["$set"]["ttl_ref"] = ttl.id
-
-        # API Spec: What is this returning? Check that it follows rules.
-        return mongo.db.monkey.update({"_id": monkey["_id"]}, update, upsert=False)
-
     # Used by monkey. can't secure.
     # Called on monkey wakeup to initialize local configuration
     @TestTelemStore.store_exported_telem