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

   ## What's changed?
   
   Adds optional SCRAM authentication to Kafka Client monitoring.
   
   - add `PLAINTEXT`, `SASL_PLAINTEXT`, and `SASL_SSL` security protocol options
   - support `SCRAM-SHA-256` and `SCRAM-SHA-512` with username and password 
credentials
   - configure the Kafka AdminClient with the selected security settings while 
preserving unauthenticated monitoring by default
   - isolate cached AdminClient connections by authentication settings and keep 
passwords out of protocol diagnostics
   - document the new settings in the English and Chinese Kafka Client 
monitoring guides
   
   Closes #4209
   
   ## Verification
   
   - `KafkaProtocolTest` (23 tests)
   - `KafkaCollectTest` (7 tests)
   - `YamlCheckScript` through the 22-module manager reactor
   - repository Markdown lint (567 files)
   
   ## Checklist
   
   - [x] I have read the [Contributing 
Guide](https://hertzbeat.apache.org/docs/community/code_style_and_quality_guide)
   - [x] 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.
   
   Not applicable: this change does not add or update an API. SCRAM validation, 
AdminClient property mapping, credential escaping, cache isolation, and 
plaintext compatibility are covered by unit tests.


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