peeringdb / peeringdb

Server code for https://www.peeringdb.com/
BSD 2-Clause "Simplified" License
340 stars 111 forks source link

UI Bug: Modifying the BFD status updates the speed for an IX connection #1589

Open netravnen opened 1 month ago

netravnen commented 1 month ago

Describe the bug

When the BFD status of an IXP connection is changed from true to false (current observation). The speed for multiple connection(s) is modified in the UI. The speed field is modified in the HTML DOM element. And not saved to the database. (Unless a subsequent edit action is done and saved without reloading the page)

https://youtu.be/DgZroTRWlMU (9 second video)

The observed behaviour seems to only apply when a network has multiple IXP connections. (This is not confirmed behaviour)

To Reproduce

Steps to reproduce the behavior:

  1. Go to a network with multiple IX connections
  2. Open Edit mode
  3. Change multiple IXP connections BFD status from true to false
  4. Save the change
  5. Observe the speed field is updated in the HTML DOM element. There is a very short noticeable delay before the update happens to the HTML DOM element.

Expected behavior

The speed field should remain the same.

Who is affected by the problem?

Users manually updating the BFD status from true to false.

What is the impact?

User confusion. Due to an unexpected behaviour of the website.

Are there security concerns?

None

Are there privacy concerns?

None

What are the proposed actions?

Investigate the root cause of the UI "bug".

What is the proposed priority?

not urgent

Provide a rationale for any/all of the above

Avoid user confusion by identifying and resolving the UI "bug".

Additional context

None