Skip to content

Conversation

@guibescos
Copy link
Contributor

@guibescos guibescos commented Jan 13, 2025

On Pythnet, until now product and price accounts were created as seeded accounts with seed product:{SYMBOL} and price:{SYMBOL}. The problem is the seed can't exceed 32 characters. Some symbols in new price feeds are very long now like Crypto.LIQUIDBERABTC/WBTC.RR and fail at creation.

There are many solutions to this:

  • slice the first 32 characters of the seed (bad because in the future there might be conflicts??? i.e. two symbols whose 24 first characters are the same but are not equal)
  • hash the full seed and take the 32 first characters of the hash (this breaks the old way of doing things and will break all existing proposals since the seeded account needs to be computed at proposal).

@vercel
Copy link

vercel bot commented Jan 13, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
api-reference ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 13, 2025 4:11pm
proposals ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 13, 2025 4:11pm
staking ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 13, 2025 4:11pm
2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
component-library ⬜️ Ignored (Inspect) Jan 13, 2025 4:11pm
insights ⬜️ Ignored (Inspect) Jan 13, 2025 4:11pm

@guibescos guibescos marked this pull request as ready for review January 13, 2025 16:22
@guibescos guibescos merged commit 4a409be into main Jan 13, 2025
9 of 10 checks passed
@guibescos guibescos deleted the fix/seed-too-long branch January 13, 2025 16:22
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.

3 participants