Skip to content

Conversation

@GuiSong01
Copy link
Contributor

Support to configure custom decryption methods for kafka configurations

  • If this is non-trivial feature, paste the links/URLs to the design doc.

  • Update the documentation to include this new feature.

  • Tests(including UT, IT, E2E) are added to verify the new feature.

  • If it's UI related, attach the screenshots below.

  • If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #10815.

  • Update the CHANGES log.

GuiSong01 added 2 commits May 26, 2023 00:12
…and fix bug `kafka-reporter-plugin-8.15.0 throw exception "java.lang.NoClassDefFoundError: com/google/gson/Gson"`
@wu-sheng
Copy link
Member

I remember there was a discussion about this, please link to the issue.

@GuiSong01
Copy link
Contributor Author

I remember there was a discussion about this, please link to the issue.

discussion is a bug kafka-reporter-plugin-8.15.0 throw exception "java.lang.NoClassDefFoundError: com/google/gson/Gson",it is fixed by Fix the gson dependency in the kafka-reporter-plugin.The PR is only about the feature skywalking-java support to decrypt password of kafka with SASL_SSL enabled

@wu-sheng wu-sheng added this to the 8.16.0 milestone May 27, 2023
@GuiSong01 GuiSong01 changed the title [Feature] Support to configure custom decryption methods for kafka configurations [Feature] Support to configure custom decode methods for kafka configurations May 31, 2023
@GuiSong01 GuiSong01 changed the title [Feature] Support to configure custom decode methods for kafka configurations [Feature] Support to configure custom decode method for kafka configurations May 31, 2023
…c/main/java/org/apache/skywalking/apm/agent/core/kafka/KafkaProducerManager.java
Copy link
Member

@wu-sheng wu-sheng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@GuiSong01 I rewrote the docs for you. Others are good for me.

Sorry for letting you wait for so long, take a long time to recovery from jetleg

@wu-sheng wu-sheng merged commit c04a714 into apache:main May 31, 2023
yangyulely pushed a commit to yangyulely/skywalking-java that referenced this pull request May 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants