[Yunohost] BMAS déployé sur `/bma`

Moul · 1 June 2023 17:24

C’est bien le cas pour le format de BASIC_MERKLED_API, par contre celui de BMAS permet la spécification d’un chemin. Je ne trouve pas la documentation de BMAS parmis :

dans duniter/lib/common-libs/constants.ts (je ne le trouve pas dans le dépôt ) :

const BMAS_REGEXP = /^BMAS( ([a-z_][a-z0-9-_.]*))?( ([0-9.]+))?( ([0-9a-f:]+))?( ([0-9]+))( (\/.+))?$/;

permet la spécification d’un chemin.

github.com

YunoHost-Apps/duniter_ynh/blob/37d9543482922c57068a86ff305a1b5aa07d124a/scripts/_common.sh#L14-L18


      
          	ynh_exec_as duniter duniter --home $datadir config --bma --ipv4 127.0.0.1 --port $port --remoteh $domain --remotep 443 --noupnp
          	ynh_exec_as duniter duniter --home $datadir config --ws2p-host 127.0.0.1 --ws2p-port 20901 --ws2p-remote-host $domain --ws2p-remote-port 443 --ws2p-remote-path "/ws2p" --ws2p-noupnp
          
          	# Add BMAS with path, remove auto-generated BMAS endpoint
          	ynh_exec_as duniter duniter --home $datadir config --addep "BMAS $domain 443 /bma" --remep "BMAS $domain 443"

La spécification L14 du port 443 créé l’endpoint BMAS incorrect sans chemin.
La L18 ajoute l’endpoint correct avec le chemin.

kimamila · 2 June 2023 14:14

Je regarde dans le code de Duniter v1, et je vois pas mal des cas où :

BMAS est ignoré: ici et ici
le path de BMAS est ignoré: iPeerDTO.getURL()](app/lib/dto/PeerDTO.ts · dev · nodes / typescript / duniter · GitLab) et p2p-candidate.ts

Du coup, à première vues, je doute que la synchro ne puisses utiliser les path BMA/BMAS …
Il y a peut-être des remplacements à la volée de la chaine BMAS par BASIC_MERKLED_API (j’en ai vu quelque part) mais dans ce cas il n’est pas sûr du tout que le procotole utilisé soit bien httpS et le noeud risque d’être marqué comme DOWN…

Je vais voir si je peux corriger cela

kimamila · 2 June 2023 16:08

J’ai publié une MR !1432 qui devrait permettre à la synchro d’utiliser les noeuds BMA/BMAS avec un chemin.

A tester !