Skip to content

doc: update OpenSSL 4 support details - #2669

Merged
George Adams (gdams) merged 2 commits into
microsoft/mainfrom
dev/dagood/doc-ossl-4-pref
Oct 28, 2026
Merged

George Adams (gdams) merged 2 commits into
microsoft/mainfrom
dev/dagood/doc-ossl-4-pref

Conversation

@dagood

Copy link
Copy Markdown
Member

@dagood
Davis Goodin (dagood) requested a review from a team as a code owner October 28, 2026 16:35
Copilot AI balanced review requested due to automatic review settings October 28, 2026 16:35
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
There may be pipelines that require an authorized user to comment /azp run to run.

Comment thread eng/doc/fips/README.md

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.

🟡 Changes recommended

The library-selection documentation omits FIPS-enabled library priority and needs to clarify that version ordering applies only among libraries with the same FIPS-enabled status.

1 open issue
What changed in this PR

Updates the Microsoft Go documentation for OpenSSL 4 support in Go 1.28, reflecting the OpenSSL 3 preference introduced in #2655 and the support documentation proposed in #2591.

Changes:

  • Lists OpenSSL 4.x and its supported providers on Linux.
  • Documents the Go 1.28 library discovery order and adds a version-history entry.
File Description
eng/​doc/​SupportedPlatforms.md Adds OpenSSL 4.x support starting with Go 1.28.
eng/​doc/​fips/​README.md Updates library discovery ordering and records OpenSSL 4 support in the changelog.

Review effort: Balanced

Comment thread eng/doc/fips/README.md Outdated
@gdams
George Adams (gdams) merged commit 37152cb into microsoft/main Oct 28, 2026
68 checks passed
@gdams
George Adams (gdams) deleted the dev/dagood/doc-ossl-4-pref branch October 28, 2026 18: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.

4 participants