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

Investigate adding eth<>oasis addr mappings from incoming api requests #778

Open
pro-wh opened this issue Oct 22, 2024 · 0 comments
Open

Comments

@pro-wh
Copy link
Collaborator

pro-wh commented Oct 22, 2024

Imported from https://app.clickup.com/t/869683dzx

Nexus currently derives its knowledge of eth<>oasis addr mappings from the evm txs it sees from on-chain. However, in some use cases a user may deposit rose into an evm (sapphire) account, and when viewing this account on the sapphire explorer Nexus will not have the corresponding eth addr info. (since consensusaccounts.deposit is not an evm tx). However, if the nexus api server sees an incoming request for an evm-style address, we could potentially have the api server update the db to include the new address. (We can derive the eth ➝ oasis addr easily, but not vice versa)

Caveats:
  1. the Nexus api is designed on purpose to be read-only, which allows us to scale it up easily to accomodate traffic. It also prevents malicious requests from corrupting the db.
  2. We have no authentication/rate limiting on the api side currently; we need to be careful about allowing these updates
  3. Luka pointed out that this is only an issue for accounts that have never interacted with Sapphire aside from consensus deposit. Once the user interacts with the evm, the mapping will be tracked by Nexus
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

No branches or pull requests

1 participant