Duansg opened a new pull request, #4225:
URL: https://github.com/apache/hertzbeat/pull/4225

   ## What's changed?
   
   Follow-up to #4220.
   
   - `resolveMaskedSecrets` silently did nothing when the submitted receiver 
had an id but no stored receiver existed (e.g. deleted concurrently). The 
subsequent `save()` then persisted the mask placeholder (like `******voJM`) as 
a real credential — and with the IDENTITY id strategy the PUT even inserted a 
new row.
   
   - Added regression tests for both `editReceiver` and `sendTestMsg`.
   
   ## Checklist
   
   - [x]  I have read the [Contributing 
Guide](https://hertzbeat.apache.org/docs/community/code_style_and_quality_guide)
   - [ ]  I have written the necessary doc or comment.
   - [x]  I have added the necessary unit tests and all cases have passed.
   
   ## Add or update API
   
   - [ ] I have added the necessary [e2e 
tests](https://github.com/apache/hertzbeat/tree/master/e2e) and all cases have 
passed.
   


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to