search

opensearch-project / alerting

📟 Get notified when your data meets certain conditions by setting up monitors, alerts, and notifications
https://opensearch.org/docs/latest/monitoring-plugins/alerting/index/
Apache License 2.0
58 stars 99 forks source link

Add validation check for doc level query name during monitor creation #1506

Closed jowg-amazon closed 2 months ago

jowg-amazon commented 3 months ago

Issue #, if available: PR related to: https://github.com/opensearch-project/common-utils/pull/630 CI's will fail until common utils PR is merged in

Description of changes: Adds validation check for doc level query name during monitor creation. Doc level query is restricted so that it may not start with: [_, +, -], contain .., or contain: [* ? < > | #]

CheckList:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license. For more information on following Developer Certificate of Origin and signing off your commits, please check here.

jowg-amazon commented 2 months ago

Security tests have been failing in previous PRs https://github.com/opensearch-project/alerting/actions/runs/8548149690/job/23421434894