Skip to content

Update FIPS user guide#2393

Open
michelle-clayton-work wants to merge 2 commits into
microsoft/mainfrom
dev/mclayton/bump-darwin-sha3
Open

Update FIPS user guide#2393
michelle-clayton-work wants to merge 2 commits into
microsoft/mainfrom
dev/mclayton/bump-darwin-sha3

Conversation

@michelle-clayton-work

@michelle-clayton-work michelle-clayton-work commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Add missing functions to guide

Copilot AI review requested due to automatic review settings July 2, 2026 19:16
@michelle-clayton-work michelle-clayton-work requested a review from a team as a code owner July 2, 2026 19:16

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 the FIPS user guide (eng/doc/fips/UserGuide.md) to document additional crypto APIs (HKDF, ML-DSA, ML-KEM, SHA-3 family) and clarify TLS behavior in newer Go versions.

Changes:

  • Added new sections documenting crypto/hkdf, crypto/mldsa, crypto/mlkem, and crypto/sha3 backend support details.
  • Expanded the TLS guidance to describe Microsoft-recommended TLS defaults and additional FIPS-mode key exchange groups.
  • Updated the document’s TOC and reference links to include the newly documented functions/APIs.

Comment thread eng/doc/fips/UserGuide.md Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@dagood

dagood commented Jul 2, 2026

Copy link
Copy Markdown
Member

Probably worth waiting and reapplying this on top of the structure added by

@dagood

dagood commented Jul 2, 2026

Copy link
Copy Markdown
Member

About waiting for #2394:

I'm fine with going ahead here and reconciling the other way, my thought is just that it might be easier to review the changes without markdown formatting being a factor.

@karianna

karianna commented Jul 5, 2026

Copy link
Copy Markdown
Member

@dagood - #2394 is now merged

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.

4 participants