Skip to content

CURLOPT_ECH.md: simplify the description language#21536

Closed
bagder wants to merge 2 commits into
masterfrom
bagder/ech-desc
Closed

CURLOPT_ECH.md: simplify the description language#21536
bagder wants to merge 2 commits into
masterfrom
bagder/ech-desc

Conversation

@bagder

@bagder bagder commented May 8, 2026

Copy link
Copy Markdown
Member

It no longer requires "a special build" of OpenSSL, just OpenSSL 4+.

Emphasize the experimental part a little clearer.

Drop the caveat for wolfSSL from the main description.

It no longer requires "a special build" of OpenSSL, just OpenSSL 4+.

Emphasize the experimental part a little clearer.

Drop the caveat for wolfSSL from the main description.
@bagder bagder requested a review from Copilot May 8, 2026 11:19
@bagder bagder marked this pull request as ready for review May 8, 2026 11:20

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates the libcurl option documentation for CURLOPT_ECH to better reflect the current state and expectations around ECH usage, with clearer emphasis on its experimental status.

Changes:

  • Adds a stronger experimental/“not for production” warning to the option description.
  • Removes the prior wording about requiring a special OpenSSL build.
  • Removes the wolfSSL HelloRetryRequest caveat from the main option description.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/libcurl/opts/CURLOPT_ECH.md Outdated
Comment thread docs/libcurl/opts/CURLOPT_ECH.md Outdated
language

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@bagder bagder closed this in d656ff9 May 8, 2026
@bagder bagder deleted the bagder/ech-desc branch May 8, 2026 14:44
outcast36 pushed a commit to greearb/curl that referenced this pull request Jun 3, 2026
It no longer requires "a special build" of OpenSSL, just OpenSSL 4+.

Emphasize the experimental part a little clearer.

Drop the caveat for wolfSSL from the main description.

Closes curl#21536
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants