DGS-22768 Update FIPS documentation for 140-3 compliance#2119
DGS-22768 Update FIPS documentation for 140-3 compliance#2119Ojasva Jain (ojasvajain) merged 8 commits intomasterfrom
Conversation
|
🎉 All Contributor License Agreements have been signed. Ready to merge. |
There was a problem hiding this comment.
Pull Request Overview
This PR updates FIPS documentation to support FIPS 140-3 compliance by clarifying the differences between FIPS 140-2 and FIPS 140-3, updating OpenSSL version requirements, and providing clearer guidance for new deployments.
- Updated OpenSSL version requirements to support both FIPS 140-2 (OpenSSL 3.0.x) and FIPS 140-3 (OpenSSL 3.1.2+)
- Added recommendation for FIPS 140-3 for new deployments due to upcoming federal procurement requirements
- Updated configuration examples and links to point to the newer OpenSSL version
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Kaushik Raina (k-raina)
left a comment
There was a problem hiding this comment.
Thanks Kipp Corman (@kcorman0) for PR!
May i know how did you test this PR with openssl 3.1.2 recommended?
Matthew Seal (MSeal)
left a comment
There was a problem hiding this comment.
You may need to do the same changes in the librdkafka repo for the non-SR compliance perspective since that repo pulls in the openssl dependency for these. I checked and it's currently using version 3.3.2 for pre-built binaries that we provide which is compatible but the readmes all have the 3.0 references instead of the 3.x. Otherwise approved for this repo
|
Kipp Corman (@kcorman0) We are currently in process of validating |
|
Ojasva Jain (@ojasvajain) is this complete? |
| @@ -1,12 +1,19 @@ | |||
| # FIPS Compliance | |||
There was a problem hiding this comment.
We need to update the same in https://github.com/confluentinc/docs-clients-confluent-kafka-python/blob/master/overview.rst#fips-compliance docs as well. This is external facing documentation for the same.
There was a problem hiding this comment.
Let's pick this up once this PR is merged.
4d3895c
Pranav Rathi (pranavrth)
left a comment
There was a problem hiding this comment.
Few minor changes.
|
Pranav Rathi (pranavrth)
left a comment
There was a problem hiding this comment.
LGTM!. Rebase the branch.





What
FIPS 140-3 (newest) requires a newer openssl version, updating documentation around this. Also went through SR dependencies and believe they're still compliant.
Probably need a +1 from clients team for non-SR aspect of this client as the change implies entire client is 140-3 compliant.
Checklist
References
JIRA:
Test & Review
Open questions / Follow-ups