Skip to content

Conversation

@rplusq
Copy link
Contributor

@rplusq rplusq commented Oct 17, 2025

Summary

  1. Upgrades turnkey_client dependency from 0.4 to 0.5
  2. Adds alloy-signer-turnkey to main alloy package re-exports

Motivation

Version 0.5.0 removes OpenSSL from the dependency tree by switching to rustls by default. This resolves cargo-deny conflicts in projects like Foundry that have workspace-wide bans on OpenSSL dependencies. This version bump is required for Foundry integration.

Additionally, the Turnkey signer should be exposed in the main alloy package for consistency with other remote signers (AWS KMS, GCP KMS). This is a separate improvement for better developer experience.

Changes

  • Bump turnkey_client from 0.4 to 0.5 in crates/signer-turnkey/Cargo.toml
  • Add alloy-signer-turnkey to workspace dependencies in root Cargo.toml
  • Add alloy-signer-turnkey dependency to crates/alloy/Cargo.toml
  • Add signer-turnkey feature flag following the pattern of signer-aws and signer-gcp
  • Add turnkey re-export to crates/alloy/src/lib.rs signers module
  • Add turnkey to eip712 feature for EIP-712 signing support

References

🤖 Generated with Claude Code

Co-Authored-By: Claude [email protected]

@rplusq rplusq changed the title chore(signer-turnkey): bump turnkey_client from 0.4 to 0.5 feat(signer-turnkey): bump to 0.5 and add to main alloy package Oct 17, 2025
## Changes
- Bump turnkey_client from 0.4 to 0.5 (removes OpenSSL dependency)
- Add alloy-signer-turnkey to workspace dependencies
- Add alloy-signer-turnkey to main alloy package
- Add signer-turnkey feature flag matching AWS/GCP pattern
- Add turnkey module re-export in signers
- Add turnkey to eip712 feature for EIP-712 signing support

This makes the Turnkey signer consistent with AWS KMS and GCP KMS signers,
and enables integration with Foundry.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
@rplusq rplusq force-pushed the deps/bump-turnkey-client branch from 8b321e9 to 05e8339 Compare October 17, 2025 23:21
Copy link
Member

@zerosnacks zerosnacks left a comment

Choose a reason for hiding this comment

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

makes sense

@github-project-automation github-project-automation bot moved this to Reviewed in Alloy Oct 22, 2025
@zerosnacks zerosnacks enabled auto-merge (squash) October 22, 2025 14:30
@zerosnacks zerosnacks merged commit 53b77b6 into alloy-rs:main Oct 22, 2025
61 of 63 checks passed
@github-project-automation github-project-automation bot moved this from Reviewed to Done in Alloy Oct 22, 2025
@zerosnacks
Copy link
Member

Thanks! Apologies for the delay

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

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants