Skip to content

DGS-22768 Update FIPS documentation for 140-3 compliance#2119

Merged
Ojasva Jain (ojasvajain) merged 8 commits intomasterfrom
kipp/fips-140-3
Jan 27, 2026
Merged

DGS-22768 Update FIPS documentation for 140-3 compliance#2119
Ojasva Jain (ojasvajain) merged 8 commits intomasterfrom
kipp/fips-140-3

Conversation

@kcorman0
Copy link
Member

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

  • [N] Contains customer facing changes? Including API/behavior changes
  • [Y] Did you add sufficient unit test and/or integration test coverage for this PR?
    • If not, please explain why it is not required

References

JIRA:

Test & Review

Open questions / Follow-ups

Copilot AI review requested due to automatic review settings October 31, 2025 20:23
@confluent-cla-assistant
Copy link

🎉 All Contributor License Agreements have been signed. Ready to merge.
Please push an empty commit if you would like to re-run the checks to verify CLA status for all contributors.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

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.

@sonarqube-confluent
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

Copy link
Member

@k-raina Kaushik Raina (k-raina) left a comment

Choose a reason for hiding this comment

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

Thanks Kipp Corman (@kcorman0) for PR!
May i know how did you test this PR with openssl 3.1.2 recommended?

Copy link
Contributor

@MSeal Matthew Seal (MSeal) left a comment

Choose a reason for hiding this comment

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

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

@ojasvajain
Copy link
Member

Kipp Corman (@kcorman0) We are currently in process of validating librdkafka for FIPS 140-3 compliance. The process is expected to complete by end of this month. Let's merge this PR then.

@kcorman0
Copy link
Member Author

Ojasva Jain (@ojasvajain) is this complete?

@@ -1,12 +1,19 @@
# FIPS Compliance

Choose a reason for hiding this comment

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

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.

Choose a reason for hiding this comment

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

Let's pick this up once this PR is merged.

Copy link
Member

@pranavrth Pranav Rathi (pranavrth) left a comment

Choose a reason for hiding this comment

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

Few minor changes.

@sonarqube-confluent
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@ojasvajain Ojasva Jain (ojasvajain) enabled auto-merge (squash) January 26, 2026 07:46
Copy link
Member

@pranavrth Pranav Rathi (pranavrth) left a comment

Choose a reason for hiding this comment

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

LGTM!. Rebase the branch.

@ojasvajain Ojasva Jain (ojasvajain) merged commit 97b0dd1 into master Jan 27, 2026
3 checks passed
@ojasvajain Ojasva Jain (ojasvajain) deleted the kipp/fips-140-3 branch January 27, 2026 05:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants