Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

behavior tree rewrite #2143

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

behavior tree rewrite #2143

wants to merge 1 commit into from

Conversation

doomedraven
Copy link
Collaborator

No description provided.

@doomedraven doomedraven added the help wanted Extra attention is needed label May 29, 2024
@doomedraven
Copy link
Collaborator Author

To prevent recursive errors on json reports, for some malware spawns a lot of processes, and we embed children into parent process

@xiangchen96
Copy link
Contributor

This works, but changing the schema of the json has some implications:

I would prefer a conf that limits the depth of the tree with a sensible default or being able to disable it (users can rebuild the tree themselves from processes if needed).

@doomedraven
Copy link
Collaborator Author

well sometime progress requires breaking changes, this rarely happens, but would be nice to handle this properly without loosing details, but yes quick fix i guess will be configurable limit, but that limit should be the same as json limit, so is only on/off as is uniq field that breaks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants