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

Seyren doesn't properly sanitize JSON #256

Open
sethdmoore opened this issue Dec 9, 2014 · 1 comment
Open

Seyren doesn't properly sanitize JSON #256

sethdmoore opened this issue Dec 9, 2014 · 1 comment
Labels

Comments

@sethdmoore
Copy link

We were notified that seyren was causing errors by posting JSON to an external API.

{"tags":["OK","seyren"],"content":" Production Application CPU Usage has entered its OK state - [summarize(graphite.path.to.app, "30sec", "avg"): 40.2125] - 2014-12-09T04:20:20.109Z - http://seyren.orgname","external_user_name":"Seyren”}

As you can see, using double quotes in the Seyren Target: field causes it to send unescaped double quote characters, breaking the JSON

Target: exclude(summarize(graphite.path.*.app,"30sec", "avg"), "seyren")

@neilprosser
Copy link
Collaborator

Which output does this happen in?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants