Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support specification of private key to claimer #350

Merged
merged 2 commits into from
Mar 14, 2024
Merged

Conversation

tuler
Copy link
Member

@tuler tuler commented Mar 13, 2024

This PR adds the option to specify CARTESI_AUTH_PRIVATE_KEY to pass to the authority-claimer in case the user have a simple private key in hand, and not a mnemonic.

The value of the private key must be a hex string of 32 bytes, without the traditional starting 0x.
i.e.: 59c6995e998f97a5a0044966f0945389dc9e86dae88c7a8412f4603b6b78690d

closes #344

@tuler tuler requested a review from gligneul March 13, 2024 18:45
@tuler tuler force-pushed the feature/private-key branch 2 times, most recently from 93fbe8a to 8589b9c Compare March 13, 2024 18:48
@tuler tuler force-pushed the feature/private-key branch from 8589b9c to f645def Compare March 13, 2024 19:28
@gligneul gligneul requested a review from renan061 March 13, 2024 20:35
Copy link
Contributor

@endersonmaia endersonmaia left a comment

Choose a reason for hiding this comment

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

It should also be possible to alternatively define a CARTESI_AUTH_PRIVATE_KEY_FILE, as we have for CARTESI_AUTH_MNEMONIC_FILE.

See: https://github.com/cartesi/rollups-node/blob/main/internal/config/generate/Config.toml#L154-L162

@tuler
Copy link
Member Author

tuler commented Mar 14, 2024

It should also be possible to alternatively define a CARTESI_AUTH_PRIVATE_KEY_FILE, as we have for CARTESI_AUTH_MNEMONIC_FILE.

See: https://github.com/cartesi/rollups-node/blob/main/internal/config/generate/Config.toml#L154-L162

Done

@tuler tuler requested a review from endersonmaia March 14, 2024 13:28
Copy link
Contributor

@renan061 renan061 left a comment

Choose a reason for hiding this comment

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

Minor comment about Doc text.

docs/config.md Show resolved Hide resolved
@endersonmaia endersonmaia self-requested a review March 14, 2024 15:25
@tuler tuler requested a review from renan061 March 14, 2024 17:17
@tuler tuler merged commit a7057b8 into main Mar 14, 2024
6 checks passed
@tuler tuler deleted the feature/private-key branch March 14, 2024 19:45
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.

Support CARTESI_AUTH_PRIVATE_KEY
3 participants