Enable Event Rules to process and send data to Scripts using Event Data

[renatoalmeidaoliveira]

Fixes: #14884

Passes Event Rule a jinja2 processed action_data to script execution as its input.

[jeremystretch]

Fixes: #14884
Fixes: #14896

Neither of these FRs have been approved before submitting this PR, and even if they had been, they would each need a separate PR. I understand that it's tempting to save time by combining work, but that also makes effective review and documentation very difficult.

I've just marked #14884 as approved and assigned it to you, but please revise the PR to remove the introduction of additional context so that we can focus on its specific goal.

[renatoalmeidaoliveira]

@jeremystretch, updated the PR, and the changes in the code reflects only the task of passing action_data to the script just like the other methods, i.e the script develop may access the input the same way he access the forms.

[jeremystretch]

@renatoalmeidaoliveira did you push your changes? I don't see any updates.

[renatoalmeidaoliveira]

@jeremystretch I may have missundertud what you asked. The changes I've made was:
1 create a method to Jinja2 process the action_data with the same context passed to the webhocks
2 passes this data to the script form to process it
3 passes the validated form to the script

You mean to just pass action_data as created by the user to the Script as a String without any processing? I think that might not be very usefull for the user because of the lack of context of the modified object

[renatoalmeidaoliveira]

I can put all code inside process_event_rules too,
Or keep process_action_data method in EventRules model but process the Script form inside it and remove the changes in the run_script method

[renatoalmeidaoliveira]

@jeremystretch updated the code to restrict the changes only to process_event_rule inside the Script context.

[aharrisson]

Is there a reason why this implementation only affect EventRuleActionChoices.SCRIPT? My use-case is to be able to use more data in a webhook payload than what I get from the default event rule "data" dict.

[renatoalmeidaoliveira]

@aharrisson imo the implementation should be the same to both Webhoocks and Scripts, giving the user more flexibility to pass data to his actions.
But anyways the Scripts it requires a different implementation to allow the user to use the data variable along with the input variable forms.
I can make further changes in that PR but I'm waiting some guidance from the maintainers.

[aharrisson]

@renatoalmeidaoliveira , Thank you for the quick reply.
Indeed keeping the capabilities consistent (as far as possible) across both action choices would make sense. I hope the maintainers can extend the boundaries of this PR to also include webhooks, otherwise the action_data form field would be confusing for event rules triggering webhooks.

[llamafilm]

Currently, if I have an event rule that triggers when an IP address is created, modified, or deleted — it's impossible for the script to know which action triggered it. Will this PR address that?

[renatoalmeidaoliveira]

@llamafilm short answer, in the implementation of this PR yes.
My idea with this PR is to enable the user to build the input passed to the Script Input vars, in a similar way the webhocks body template does

[github-actions]

This PR has been automatically marked as stale because it has not had recent activity. It will be closed automatically if no further action is taken.

[arthanson]

@renatoalmeidaoliveira after reviewing with Jeremy, the jinja2 processing shouldn't be here, the data should just be passed as a json blob without going through jinja2. Can you please update the PR for this?