Skip to content

docs: Update ecdsa messaging#486

Merged
theekrystallee merged 9 commits intomainfrom
update-ecdsa-messaging
Mar 25, 2026
Merged

docs: Update ecdsa messaging#486
theekrystallee merged 9 commits intomainfrom
update-ecdsa-messaging

Conversation

@ty-swirldslabs
Copy link
Contributor

Description:

This PR updates key type documentation across three pages to recommend ECDSA (secp256k1) for all new accounts and applications, reflecting its advantages for EVM and smart contract compatibility.

  • Update keys-and-signatures.mdx to recommend ECDSA as the default key type, restructure the key type section with dedicated ECDSA and Ed25519 subsections, add accurate EVM alias derivation details, add a code example showing setECDSAKeyWithAlias() for account creation, expand the comparison table, and add new FAQ entries for key type selection and EVM alias explanation
  • Update generate-a-new-key-pair.mdx to list ECDSA first and update all code examples to use ECDSA key generation methods, moving ED25519 to a secondary section
  • Update import-an-existing-key.mdx to replace all ED25519 key import examples with ECDSA equivalents across Java, JavaScript, Go, and Rust

Notes for reviewer:

The EVM alias description on keys-and-signatures.mdx is taken verbatim from the HIP/protocol spec: the alias is the rightmost 20 bytes of the 32-byte Keccak-256 hash of the ECDSA public key, per the Ethereum Yellow Paper, with the recovery ID excluded. The code example uses setECDSAKeyWithAlias() to clarify that the alias must be explicitly set at account creation, it is not set automatically.

@ty-swirldslabs ty-swirldslabs requested review from a team as code owners March 24, 2026 22:18
@mintlify
Copy link
Contributor

mintlify bot commented Mar 24, 2026

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
hedera-docs 🟢 Ready View Preview Mar 24, 2026, 10:25 PM

@theekrystallee theekrystallee changed the title Update ecdsa messaging docs: Update ecdsa messaging Mar 25, 2026
Co-authored-by: krystal <56278409+theekrystallee@users.noreply.github.com>
Signed-off-by: Ty <146887393+ty-swirldslabs@users.noreply.github.com>
Co-authored-by: krystal <56278409+theekrystallee@users.noreply.github.com>
Signed-off-by: Ty <146887393+ty-swirldslabs@users.noreply.github.com>
Co-authored-by: krystal <56278409+theekrystallee@users.noreply.github.com>
Signed-off-by: Ty <146887393+ty-swirldslabs@users.noreply.github.com>
ty-swirldslabs and others added 2 commits March 25, 2026 09:45
Co-authored-by: krystal <56278409+theekrystallee@users.noreply.github.com>
Signed-off-by: Ty <146887393+ty-swirldslabs@users.noreply.github.com>
Co-authored-by: krystal <56278409+theekrystallee@users.noreply.github.com>
Signed-off-by: Ty <146887393+ty-swirldslabs@users.noreply.github.com>
Copy link
Member

@theekrystallee theekrystallee left a comment

Choose a reason for hiding this comment

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

LGTM. thank you!

@theekrystallee theekrystallee merged commit 5c065f1 into main Mar 25, 2026
14 checks passed
@theekrystallee theekrystallee deleted the update-ecdsa-messaging branch March 25, 2026 15: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.

2 participants