davsclaus commented on code in PR #27419:
URL: https://github.com/apache/camel/pull/27419#discussion_r4192828453


##########
components/camel-spiffe/src/main/docs/spiffe-component.adoc:
##########
@@ -155,6 +155,14 @@ verified SPIFFE ID:
 
 [source,java]
 ------------------------------------------------------------
+// the server must REQUIRE a client certificate, or the policy has no verified 
peer to authorize
+SpiffeSSLContextParameters serverSsl = new SpiffeSSLContextParameters();
+serverSsl.setAcceptedSpiffeIds("spiffe://example.org/frontend");

Review Comment:
   Optional: worth one sentence that this TLS-level allow-list must contain 
every ID that any route's `SpiffeSecurityPolicy` accepts, otherwise the 
handshake rejects the peer before the policy runs. That matters when several 
routes with different policies share one server.



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

Reply via email to