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

   ### Before Creating the Bug Report
   
   - [x] I have searched the [open 
issues](https://github.com/apache/rocketmq-dashboard/issues) and found no 
similar issue.
   
   ### What Happened
   
   `docs/api-spec.md` 3.3 documents the instance update body as `{id, name, 
type, endpoint}` with all
   four fields required. The endpoint takes `UpdateInstanceDTO`, where only 
`instanceId` is required
   (`@NotBlank` — the instance name with a numeric primary-key fallback); 
`name`, `type` and `endpoint`
   are optional partial updates, `remark` and `adminCredentialRef` are accepted 
but undocumented, and no
   field named `id` exists in the DTO.
   
   Additionally, 4.14 / 4.15 (K8s certificate update & delete) type the `id` as 
a string while the DTOs
   declare `Long`.
   
   ### Expected Behavior
   
   3.3 should list `instanceId` as the only required field with the optional 
partial-update fields, and
   the certificate ids should be typed as numbers.
   


-- 
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