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

Keyring management for icq-relayer #1656

Open
joe-bowman opened this issue Jun 13, 2024 · 1 comment
Open

Keyring management for icq-relayer #1656

joe-bowman opened this issue Jun 13, 2024 · 1 comment
Assignees
Labels
enhancement New feature or request tool: icq-relayer

Comments

@joe-bowman
Copy link
Contributor

joe-bowman commented Jun 13, 2024

Summary

icq-relayer, prior to v1 used the lens library to handle key management

Since removing the deprecated lens, we need to add in keyring management ourselves. Currently, the config file references a seed file. This is useful in cases where we can store the seed in a proper secrets management solution and mount this on container start, but otherwise this is far from an ideal solution.

As such, we need to add the ability to hook into and leverage the existing Cosmos keyring logic, but also maintain the ability to choose to use the external seed phrase where this is relevant and safe to do so.

Note: this should be built on and PR'd to the icq/v1.0.x branch, not main.

@joe-bowman joe-bowman reopened this Jun 13, 2024
@joe-bowman joe-bowman added the enhancement New feature or request label Jun 13, 2024
@hoank101
Copy link
Contributor

hi @joe-bowman please assign this issues to me

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request tool: icq-relayer
Projects
None yet
Development

No branches or pull requests

2 participants