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

Support multiline alert log in integrator messages #3

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

jmpesp
Copy link
Contributor

@jmpesp jmpesp commented Mar 1, 2016

Fill the alertlog integrator variable using every line of alert log, not just the first one.

Also increases log limit sizes.

`sh` supports multiline variables like so:

    variable='test
    data
    multiline'

Fill the alertlog integrator variable using every line of alert log. not
just the first one.
Existing integrations use a shell script to POST data and a previous
commit changed the `alertlog` variable so it contains the full multiline
alert log. This requires each line to be sanitized, otherwise the shell
variable definition will be incorrect. Iterate over log output and
sanitize each line.

In the Slack integration, `curl` has been changed to use `--data-binary`
so it will _not_ strip out newlines and carriage returns when POSTing
data.
Number of open ports in netstat output is unusually large on our
instances so bump up the log_size limit.
@jmpesp jmpesp force-pushed the support_multiline_alert_log_in_integrator_messages branch from 49839cf to a6d545d Compare February 12, 2021 21:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant