yx9o opened a new issue, #613:
URL: https://github.com/apache/rocketmq-dashboard/issues/613

   ## Problem
   
   `POST /api/instances/update` applies non-null `name` and `endpoint` values 
without checking whether they are blank.
   
   An existing instance can therefore be saved with whitespace-only required 
fields, which is inconsistent with instance creation validation.
   
   ## Expected behavior
   
   - Reject non-null blank instance names with HTTP 400.
   - Reject non-null blank instance endpoints with HTTP 400.
   - Leave the existing instance unchanged when validation fails.
   - Continue treating null fields as omitted fields during partial updates.
   - Preserve normal updates with valid values.
   
   ## Proposed fix
   
   Validate the supplied name and endpoint before modifying the existing 
instance, and add regression tests covering blank fields and mutation 
prevention.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to