Target Directories from Ransomware plugin are broken about monkey HOT 4 CLOSED

Sounds like #1 is perhaps the best approach. There may be other reasons in the future we would want a custom schema. Let's just make sure the build script prints a visible warning (use colors if possible) that warns when the schema is not regenerated.

from monkey.

Sounds like #1 is perhaps the best approach. There may be other reasons in the future we would want a custom schema. Let's just make sure the build script prints a visible warning (use colors if possible) that warns when the schema is not regenerated.

Something like this?

from monkey.

The problem started when we updated the plugin build script to generate the config schema based on the options model. Before this, the schemas were generated manually. Now the build script uses pydantic's model_json_schema to generate the schema.

The ransomware target directory options are nullable types. The manually generated schema modeled this by using ["null", "string"] as the type of the field. The pydantic generated schema uses anyOf with one of the options as type: null. This anyOf is rendered by RJSF as a dropdown list.

Pydantic issue discussing anyOf for nullable types
PR adding RJSF's support for nullable types
RJSF documentation on nullable types

from monkey.

Potential solutions:

1. Provide a manually generated schema for Ransomware, and update the build script to only generate the config schema if one doesn't already exist
2. Use a custom generator that flattens out the anyOf when generating the schema
3. Flatten out the anyOf in the Island UI when retrieving the schema from the API
4. Submit a PR to RJSF to handle this anyOf case

from monkey.