ppalaga commented on code in PR #6241:
URL: https://github.com/apache/camel-quarkus/pull/6241#discussion_r1674209331


##########
extensions/crypto/runtime/src/main/doc/usage.adoc:
##########
@@ -0,0 +1,23 @@
+=== FIPS
+
+When running `crypto` extension on FIPS enabled system the BC-FIPS has to be 
utilized.

Review Comment:
   > 1 - In non-FIPS environment all worjs out of the box.
   
   I'd vote for being more explaining even for non-FIPS. We should note the that
   
   * BC dependency was removed from crypto in Camel 3.8.0. 
   * If users need algorithms or hash functions provided by BC, they need to 
add what? - camel-quarkus-support-bouncycastle AND bcprov? 
camel-quarkus-support-bouncycastle will do what? - register BC as a Security 
Provider and add native support (correct me if I am wrong)?
   * When adding other non-default security providers to class path, users 
should also register them.
   
   > 2 - In the FIPS environment
   
   Sounds right.



-- 
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: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to