"tower_notification_template" module fails when type is webhook and the custom messages is used with "message" and is successful if used with "body"

SWADESNA commented 3 years ago

ISSUE TYPE

Bug Report

SUMMARY

When we specify the custom message like below in the tower_notification_template, it seems to work perfectly and when notification is associated with any Job Template run, the desired notification format is sent as well in the destination webhook.

      messages:
        started:
          body: |-
            {
              "tower_source"  : "Ansible Tower Dev",
              "id"            : "{{ job.id }}"
            }

But when the same is configured using "message" instead of "body", the playbook seems to fail.
We need to more clarification on the tower_notification_template documentation here on the formats for message/body that needs to be used.
Also the difference between message/body.

ENVIRONMENT

Ansible version: 2.9.13
Operating System: RHEL
Ansible Tower
STEPS TO REPRODUCE

Below sample can be used for testing while creating Customized messages:

      messages:
        started:
          message: |-
            {
              "tower_source"  : "Ansible Tower Dev",
              "id"            : "{{ job.id }}"
            }

EXPECTED RESULTS

The "message" should work like the "body"
ACTUAL RESULTS

It fails with below:

FAILED! => {"changed": false, "msg": "Unable to create notification_template webhook1 notification: {'messages': ['Messages cannot contain newlines (found newline in started event)']}"}

ADDITIONAL INFORMATION

The below attached image for successful creation using "body". body_success

wenottingham commented 3 years ago

Marking as an enhancement for better module docs.

lucab85 commented 3 years ago

+1 for case 02969503

lucab85 commented 3 years ago

Please consider a UI update to set the "body" field. At the moment it's only possible to set the "message" field in the notification template area: Screenshot from 2021-06-23 12-24-37

azrdev commented 1 year ago

Please consider a UI update to set the "body" field. At the moment it's only possible to set the "message" field in the notification template area:

this seems to be implemented in AWX 21.1.0 which allows to set * message bodys

ansible / awx