RDoc-2530 added the Kafka ETL API and Studio pages "setup workflow" and "prerequisites for a secure Kafka server" sections#2468
Open
reebhub wants to merge 3 commits into
Open
RDoc-2530 added the Kafka ETL API and Studio pages "setup workflow" and "prerequisites for a secure Kafka server" sections#2468reebhub wants to merge 3 commits into
reebhub wants to merge 3 commits into
Conversation
…afka server" section (v6.2-v7.2)
…nd "prerequisites" sections (6.2-7.2)
poissoncorp
reviewed
Jun 9, 2026
| // Authentication via SASL/PLAIN over SSL/TLS | ||
| ConnectionOptions = new Dictionary<string, string> | ||
| { | ||
| { "security.protocol", "SASL_SSL" }, |
Contributor
There was a problem hiding this comment.
please describe the options used here - at least a bit - what each one means and what's the expected/recommended value set
poissoncorp
reviewed
Jun 9, 2026
| KafkaConnectionSettings = new KafkaConnectionSettings | ||
| { | ||
| BootstrapServers = "broker.example.com:9092", | ||
| UseRavenCertificate = true |
Contributor
There was a problem hiding this comment.
Documentation should address the concern that RavenDB's cluster server certificate has only server-auth EKU, so it cannot be used for client communication. Are we using different one here?
poissoncorp
reviewed
Jun 9, 2026
|
|
||
| If enabled, RavenDB will export to the target machine/s the cluster-wide | ||
| certificate defined during setup, and secure its connection with them. | ||
| If enabled, the Kafka connection runs over SSL/TLS, and RavenDB authenticates to Kafka using the cluster-wide certificate defined during setup. |
poissoncorp
reviewed
Jun 9, 2026
| * **Note**, however, that to complete the process you still need to register RavenDB's | ||
| exported certificate in [Kafka's truststore](https://kafka.apache.org/documentation/streams/developer-guide/security.html) | ||
| on the target machine/s. | ||
| * **Note**, however, that to complete the process you still need to register RavenDB's cluster-wide certificate |
Contributor
There was a problem hiding this comment.
let's highlight it - extremely important
poissoncorp
reviewed
Jun 9, 2026
| exported certificate in [Kafka's truststore](https://kafka.apache.org/documentation/streams/developer-guide/security.html) | ||
| on the target machine/s. | ||
| * **Note**, however, that to complete the process you still need to register RavenDB's cluster-wide certificate | ||
| in [Kafka's truststore](https://kafka.apache.org/documentation/streams/developer-guide/security.html) on the target machine/s. |
Contributor
There was a problem hiding this comment.
do we want a word or two about kafka truststore, or it'd be a domain overstretch (documenting Kafka)?
Contributor
Author
There was a problem hiding this comment.
right
added a few words in the same warning frame
poissoncorp
requested changes
Jun 9, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Issue link
RDoc-2530
Additional description
Type of change
/templatesor readme)Changes in docs URLs
/scripts/redirects.jsonfile, setDocuments MovedPR label)Changes in UX/UI