Skip to content
Snippets Groups Projects

idty_by_username

idty_by_username
tuxmain/duniter-gva:idty_by_username into master
1 unresolved thread
Open Pascal Engélibert requested to merge tuxmain/duniter-gva:idty_by_username into master
1 unresolved thread

Add IdtyByUsername GVA query.

J'ai dû ajouter la pubkey dans gva_identities pour le revert uniquement.

@librelois

Edited by Pascal Engélibert

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Éloïs
  • Éloïs
    Éloïs @librelois started a thread on the diff
    • db/src/lib.rs
    67 67 GvaUtxoIdDbV1,
    68 68 SourceAmountValV2
    69 69 ],
    70 ["idty_by_username", IdtyByUsername, String, PubKeyValV2],
    • Éloïs
      Éloïs @librelois
      Owner

      Une optimisation possible: Remplacer String par ArrayString<100>

    • Pascal Engélibert
      Pascal Engélibert @tuxmain
      Author Guest

      ArrayString n'impl pas les bons traits, donc il faudrait faire un wrapper... et il a une taille constante, veut-on vraiment prendre 100 octets par username quand ils font en moyenne moins de 10 ?

      Si on veut optimiser ça, ça ne serait pas plutôt en utilisant un type où la longueur en u8 ?

    • Éloïs
      Éloïs @librelois
      Owner

      Je parle d'optimiser le temps d'exécution, pas la mémoire. ArrayString permet d'éviter les heap allocations.

      Edited by Éloïs
    • Please register or sign in to reply
  • Éloïs
    Éloïs @librelois
    Owner

    J'ai dû ajouter la pubkey dans gva_identities pour le revert uniquement.

    Non tu n'en à pas besoin. La pubkey est dans le bloc que tu revert, ty y à accès.

    @tuxmain pour être insensible à la casse: insère la clé au format uppercase. Et pour fournir le vrai username il te faudra get l'identité dans BcV2 ;)

  • Pascal Engélibert added 1 commit

    added 1 commit

    • 20b7515d - feat(db+indexer+gql): idty_by_username

    Compare with previous version

    • Please register or sign in to reply