I have opened issue #1442 to improve the response time of the BMA /tx/history/:pubkey/pending endpoint. Cesium makes this request every time the wallet is refreshed. Users may be waiting for a transaction sent by another user, making it necessary for us to retrieve this data from the network. Howe…

versionMin is the minimum document version for a transaction to be included in the result. medianTime is not used. (see function get_pending_txs) I’m not sure what would be the simpler or cleaner way, but if needed I can add something like get_pending_transactions_for_bma.

Merci @tuxmain pour ta réponse. J’ai ajouté deux fonction au module Rust duniter-gva : get_written_transactions_for_bma() qui remplit uniquement sent/received get_pending_transactions_for_bma() : qui remplit uniquement sending/pending j’ai modifié get_transactions_history_for_bma() pour qu’il app…

J’ai fini 2 MR : sur duniter-core : [enh] add `get_written_transactions_for_bma()` and `get_pending_transactions_for_bma()` (!2) · Merge requests · nodes / rust / duniter-core · GitLab sur duniter-gva : feat(bma) add `get_written_transactions_for_bma()` and `get_pending_transactions_for_bma()` - …

C’est bon pour moi. (je n’ai pas les droits sur les dépôts donc je ne peux ni approuver ni merger) Ma MR sur duniter-core peut éventuellement éviter des crashes aux nœuds GVA mais je ne sais pas/plus si c’est déjà arrivé en production. Celles sur duniter-gva sont abandonnées, à moins que quelqu’un …

[image] tuxmain: C’est bon pour moi. (je n’ai pas les droits sur les dépôts donc je ne peux ni approuver ni merger) J’ai les droits. Je peux lancer les merge si j’ai ton feu vert (je suis bien incapable de donner mon avis sur la MR elle-même :slight_smile: ).

[image] tuxmain: C’est bon pour moi. (je n’ai pas les droits sur les dépôts donc je ne peux ni approuver ni merger) J’ai commencé à faire l’oxydation, et j’ai un soucis sur get_written_transactions_for_bma_with_range(). j’ai l’impression que le end_block n’est pas inclus dans la recherche. …

en effet les ranges en Rust a..b incluent a et non b. Il faut faire end_block+1 pour l’inclure. Edit: ou plutôt end_block.saturating_add(1) sinon il y aura un panic pour end_block=u32::MAX.

[image] tuxmain: (je n’ai pas les droits sur les dépôts donc je ne peux ni approuver ni merger) Voilà qui est corrigé, je t’ai invité dans les Owners du groupe nodes/rust.

Merci @tuxmain du coup je m’en suis sorti pour les TX par plage de blocs. Ouf. Tant que j’y suis, je vais ajouter à ma MR une fonction Rust pour retrouver les TX par plages de times. J’ai qu’il en existe une pour les pages graphql, mais elle utilise le median_time. Je ne sais plus pour BMA si c’éta…

[image] kimamila: Tant que j’y suis, je vais ajouter à ma MR une fonction Rust pour retrouver les TX par plages de times. Voila, j’ai eu du mal mais ca fonctionne : toutes les requetes suivantes ont été optimisé : /tx/history/:pubkey/pending /tx/history/:pubkey/blocks/:from/:to /tx/history/…