Skip to content

Conversation

@elizabethengelman
Copy link
Collaborator

@elizabethengelman elizabethengelman commented Mar 21, 2025

What

Todo:

  • make sure that stellar tx new set-options --set-revocable --source mykey2 works (i.e. we should be able to sign newly created tx with a secure store key)
  • make sure that tx sign supports secure store

Why

This was previously broken. The current workaround is to use --build-only and then pipe that output into stellar tx sign which does work. This works because stellar tx sign uses a different code path (config::sign_with::Args.sign_tx_env) that gets the signer from the Secret, instead of trying to get the private key directly.

Known limitations

@github-project-automation github-project-automation bot moved this to Backlog (Not Ready) in DevX Mar 21, 2025
@elizabethengelman elizabethengelman force-pushed the fix/secure-storage-signing branch from e8bc02f to b8a4dd3 Compare April 1, 2025 15:39
@elizabethengelman elizabethengelman force-pushed the fix/secure-storage-signing branch from 97a87c3 to 917f129 Compare April 9, 2025 13:12
@elizabethengelman elizabethengelman changed the title wip Fix signing with a secure store key Apr 9, 2025
@elizabethengelman elizabethengelman marked this pull request as ready for review April 10, 2025 21:56
@elizabethengelman elizabethengelman requested a review from a team April 14, 2025 14:11
@elizabethengelman elizabethengelman moved this from Backlog (Not Ready) to Needs Review in DevX Apr 16, 2025
@elizabethengelman elizabethengelman force-pushed the fix/secure-storage-signing branch from ca84a24 to eb12792 Compare April 22, 2025 15:04
@elizabethengelman elizabethengelman force-pushed the fix/secure-storage-signing branch 7 times, most recently from 8052b6b to f292ef1 Compare April 22, 2025 22:00
@elizabethengelman elizabethengelman self-assigned this Apr 23, 2025
@elizabethengelman elizabethengelman force-pushed the fix/secure-storage-signing branch from 7d04304 to 6b288a4 Compare April 23, 2025 17:08
@elizabethengelman elizabethengelman enabled auto-merge (squash) April 24, 2025 15:25
@elizabethengelman elizabethengelman removed the request for review from leighmcculloch April 29, 2025 18:58
@elizabethengelman elizabethengelman merged commit fc64fff into main Apr 29, 2025
34 checks passed
@elizabethengelman elizabethengelman deleted the fix/secure-storage-signing branch April 29, 2025 18:58
@github-project-automation github-project-automation bot moved this from Needs Review to Done in DevX Apr 29, 2025
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.

Fix signing with a secure store key with stellar tx new ...

4 participants