Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[internal/kafka] define a common validation function for kafka authentication #27674

Closed

Conversation

sakulali
Copy link
Contributor

@sakulali sakulali commented Oct 15, 2023

Description:
Define a common validation function for kafka authentication, which can be reused by kafkametricsreceiver, kafkareceiver and kafkaexporter. In addition, extract redundant validation logic between the validateSASLConfig and configureSASL functions.

Link to tracking Issue:
#27486

Testing:
make chlog-validate
go test for internal/kafka
go test for kafkaexporter
go test for kafkametricsreceiver
go test for kafkareceiver

Documentation:

@github-actions
Copy link
Contributor

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@github-actions github-actions bot added the Stale label Oct 30, 2023
@sakulali sakulali force-pushed the add-kafka-validate-authentication branch from 21203ce to 931b186 Compare October 31, 2023 06:34
@github-actions github-actions bot removed the Stale label Nov 1, 2023
Copy link
Contributor

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@github-actions github-actions bot added the Stale label Nov 15, 2023
@sakulali sakulali force-pushed the add-kafka-validate-authentication branch from 931b186 to 9269f73 Compare November 18, 2023 07:13
@github-actions github-actions bot removed the Stale label Nov 19, 2023
Copy link
Contributor

github-actions bot commented Dec 3, 2023

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@github-actions github-actions bot added the Stale label Dec 3, 2023
@sakulali sakulali force-pushed the add-kafka-validate-authentication branch from 9269f73 to 49b5f01 Compare December 7, 2023 05:28
@github-actions github-actions bot removed the Stale label Dec 8, 2023
@sakulali sakulali force-pushed the add-kafka-validate-authentication branch from 49b5f01 to 91b70ec Compare December 21, 2023 13:45
Copy link
Contributor

github-actions bot commented Jan 5, 2024

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@github-actions github-actions bot added the Stale label Jan 5, 2024
Copy link
Contributor

Closed as inactive. Feel free to reopen if this PR is still being worked on.

@github-actions github-actions bot closed this Jan 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants