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

[Synthetics][Uptime] Add SANs and Monitor info to alert #136358

Closed
andrewvc opened this issue Jul 14, 2022 · 4 comments
Closed

[Synthetics][Uptime] Add SANs and Monitor info to alert #136358

andrewvc opened this issue Jul 14, 2022 · 4 comments
Labels
enhancement New value added to drive a business result Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability

Comments

@andrewvc
Copy link
Contributor

andrewvc commented Jul 14, 2022

Per this discuss thread it really would make sense to include context variables in TLS alerts for SANs and possible monitor info as well. Right now we heavily lean on Common Name which has been deprecated for a while, though is often still a helpful field.

As a first step it'd be great to add a context.SubjectAlternativeNames variable, which would make it more clear to those receiving alerts which domains were potentially affected.

As a stretch goal it'd be good to add something like context.MonitorNames and context.MonitorUrls as well, since users may only be interested in the subset of domains that they actually monitor. This is important when using hosts such as cloudflare, which share certs among customers, meaning that many of the SANs may be irrelevant or confusing to the user.

@andrewvc andrewvc added enhancement New value added to drive a business result Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability labels Jul 14, 2022
@elasticmachine
Copy link
Contributor

Pinging @elastic/uptime (Team:uptime)

@paulb-elastic
Copy link
Contributor

@andrewvc is this a dupe of elastic/uptime#464?

@paulb-elastic
Copy link
Contributor

@andrewvc is this a dupe of elastic/uptime#464?

@andrewvc bump ^^

Also, I have moved this to post 1.0 (lmk if that's not right)

@andrewvc
Copy link
Contributor Author

Yes, it is a dupe, closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New value added to drive a business result Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability
Projects
None yet
Development

No branches or pull requests

3 participants