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] Monitor add route allows duplicate monitor names #175759

Closed
awahab07 opened this issue Jan 28, 2024 · 3 comments
Closed

[Synthetics] Monitor add route allows duplicate monitor names #175759

awahab07 opened this issue Jan 28, 2024 · 3 comments
Labels
bug Fixes for quality problems that affect the customer experience Synthetics Team:obs-ux-management Observability Management User Experience Team

Comments

@awahab07
Copy link
Contributor

Kibana version: 8.12.0

Elasticsearch version:

Server OS version:

Browser version:

Browser OS version:

Original install method (e.g. download page, yum, from source, etc.):
Tested on 8.13.0-SNAPSHOT

Describe the bug:
Although the Synthetics UI validates the monitor for duplicate names but the server route does not. So if there's some re-trying due to how the browser or the network behaves, it may recreate many duplicate monitors.

Steps to reproduce:

  1. Go "Create monitor" page on Synthetics UI and create an HTTP monitor with simplest configuration.
  2. Before pressing save, open browser Network panel in dev tools to track the POST request to /internal/synthetics/service/monitors route.
  3. Hit save and "Copy as fetch" the sent request.
  4. Issue the same request from browser console a few time.
  5. Note that the requests go successful resulting in many duplicate monitors.

Expected behavior:
The route should return Baa Request with error message that a monitor with the same name has already been created.

Screenshots (if relevant):
Screenshot 2024-01-28 at 15 27 00

Errors in browser console (if relevant):

Provide logs and/or server output (if relevant):

Any additional context:
The issue has been reported on forum.

@awahab07 awahab07 added bug Fixes for quality problems that affect the customer experience Synthetics Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team labels Jan 28, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services)

@smith smith added Team:obs-ux-management Observability Management User Experience Team and removed Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team labels Jul 8, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-management-team (Team:obs-ux-management)

@dominiqueclarke
Copy link
Contributor

Resolved by 29e8394

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience Synthetics Team:obs-ux-management Observability Management User Experience Team
Projects
None yet
Development

No branches or pull requests

4 participants