Skip to content

Fix typo in TLS cipher docs: POLY1205 → POLY1305#4976

Closed
Copilot wants to merge 2 commits intomainfrom
copilot/fix-tls-cipher-name
Closed

Fix typo in TLS cipher docs: POLY1205 → POLY1305#4976
Copilot wants to merge 2 commits intomainfrom
copilot/fix-tls-cipher-name

Conversation

Copy link
Copy Markdown

Copilot AI commented Apr 13, 2026

ECDHE-RSA-CHACHA20-POLY1205 is not a valid cipher suite — the correct suffix is POLY1305. Users copying this value from the docs would get SSL_CTX_set_cipher_list:no cipher match errors. The ECDSA variant (ECDHE-ECDSA-CHACHA20-POLY1305) on the adjacent line was already correct; only the RSA variant had the typo.

Description

Single-character typo fix in the APM Server SSL cipher suites reference table.

  • Fixed: ECDHE-RSA-CHACHA20-POLY1205ECDHE-RSA-CHACHA20-POLY1305 in docs/en/observability/apm/shared-ssl-config.asciidoc
  • All other cipher entries in the table were validated and are correct.

Documentation sets edited in this PR

Check all that apply.

  • Stateful (docs/en/observability/*)
  • Serverless (docs/en/serverless/*)
  • Integrations Developer Guide (docs/en/integrations/*)
  • None of the above

Checklist

  • Product/Engineering Review
  • Writer Review

Follow-up tasks

Select one.

  • This PR does not need to be ported to another doc set because:
    • The concepts in this PR only apply to one doc set (serverless or stateful)
    • The PR contains edits to both doc sets (serverless and stateful)
  • This PR needs to be ported to another doc set:
    • Port to stateful docs: <link to PR or tracking issue>
    • Port to serverless docs: <link to PR or tracking issue>

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://api.github.com/repos/elastic/observability-docs/pulls/4976
    • Triggering command: /usr/bin/curl curl -s -X PATCH -H Authorization: token ****** -H Accept: application/vnd.github.v3&#43;json REDACTED -d {&#34;base&#34;: &#34;8.x&#34;, &#34;draft&#34;: false} sh -c &#34;mcp-server-play--exclude-hidden=fetch (http block)
    • Triggering command: /usr/bin/curl curl -s -X PATCH -H Authorization: token ****** -H Accept: application/vnd.github.v3&#43;json REDACTED -d {&#34;base&#34;: &#34;8.x&#34;, &#34;draft&#34;: false} git conf�� get --global t http.https://gitgit (http block)

If you need me to access, download, or install something from one of these locations, you can either:

@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Apr 13, 2026

This pull request does not have a backport label. Could you fix it @Copilot? 🙏
To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-/d./d is the label to automatically backport to the /d./d branch. /d is the digit
  • backport-active-all is the label that automatically backports to all active branches.
  • backport-active-8 is the label that automatically backports to all active minor branches for the 8 major.
  • backport-active-9 is the label that automatically backports to all active minor branches for the 9 major.
    NOTE: backport-skip has been added to this pull request.

@mergify mergify bot added the backport-skip Skip notification from the automated backport with mergify label Apr 13, 2026
Copy link
Copy Markdown
Member

@carsonip carsonip left a comment

Choose a reason for hiding this comment

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

should not create a new file. Just fix any typos if exist

@carsonip carsonip closed this Apr 13, 2026
Copilot AI changed the title [WIP] Fix incorrect cipher name in TLS documentation Fix typo in TLS cipher docs: POLY1205 → POLY1305 Apr 13, 2026
Copilot stopped work on behalf of carsonip due to an error April 13, 2026 17:54
Copilot AI requested a review from carsonip April 13, 2026 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-skip Skip notification from the automated backport with mergify

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Incorrect cipher name in TLS documentation: POLY1205 should be POLY1305

2 participants