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

Automatically rotate Noble deposit addresses #4873

Open
zbuc opened this issue Sep 30, 2024 · 0 comments · May be fixed by #4878
Open

Automatically rotate Noble deposit addresses #4873

zbuc opened this issue Sep 30, 2024 · 0 comments · May be fixed by #4878

Comments

@zbuc
Copy link
Member

zbuc commented Sep 30, 2024

Is your feature request related to a problem? Please describe.
Building on #4857, now that we have a basic implementation of Noble forwarding address registration, they should be rotated automatically on use.

Describe the solution you'd like

image

We can keep this simple and use the existing work from #4857 to implement speculative registration:

To figure out the next sequence number, assuming the view service doesn't have it available, submit a register tx starting at index 0 until you get a "account doesn't exist" error, this is the next one in the sequence.

To poll: submit the register tx repeatedly until you get the "successfully registered" message instead of the "doesn't exist" error.

For now, the sequence number should be stored in the view service. We can figure out how to expose it to Prax later.

@github-actions github-actions bot added the needs-refinement unclear, incomplete, or stub issue that needs work label Sep 30, 2024
@zbuc zbuc linked a pull request Oct 2, 2024 that will close this issue
1 task
@aubrika aubrika removed the needs-refinement unclear, incomplete, or stub issue that needs work label Oct 7, 2024
@aubrika aubrika added this to Penumbra Oct 7, 2024
@github-project-automation github-project-automation bot moved this to Backlog in Penumbra Oct 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Backlog
Development

Successfully merging a pull request may close this issue.

2 participants