caronc / apprise

Apprise - Push Notifications that work with just about every platform!
https://hub.docker.com/r/caronc/apprise
BSD 2-Clause "Simplified" License
12.09k stars 421 forks source link

OpsGenie Unexpected Concatenation of Title and Body, and Removal of Title #654

Closed KentonParton closed 2 years ago

KentonParton commented 2 years ago

:mega: Notification Service(s) Impacted

OpsGenie

:lady_beetle: Describe the bug

When providing a title and body, the OpsGenie Notification Service concatenates the title and body fields into a title_body variable, here. This is an unexpected behaviour. If the user of the library wanted the body included in the title, they should control that.

A few lines later here, the title, provided by the user, is removed from title_body if it exceeds 130 characters resulting in the desired title not appearing at all but rather a shortened body.

:bulb: Screenshots and Logs

Here we can see that title, MY DESIRED TITLE??? has been concatenated with body to form message in the payload.

Screenshot 2022-08-28 at 16 28 24

Here, a few lines later, we can see that the title has been removed:

Screenshot 2022-08-28 at 16 28 48

:computer: Your System Details:

:crystal_ball: Additional context In my opinion, title_body should be removed. This will allow the user of the library to decide whether to concatenate the title and body.

caronc commented 2 years ago

Closing this off as you fixed it :). Thank you!