Tu as une double parenthèse ligne 20, ce qui fait planter le script.
Tu auras ensuite un autre problème en cliquant sur le bouton : Cannot GET /currency-monit/
. En effet dans le cas présent tu crées un nouveau serveur HTTP sur le port 10500, or l’URL ouverte par le bouton est relative à Duniter UI, donc :
- le lien pointe vers http://localhost:9220/currency-monit/
- duniter-currency-monit sert http://localhost:10500/currency-monit/
Et cela vient d’ici :
https://github.com/duniter/duniter-currency-monit/blob/dev-module/index.js#L63-L67
Avec les modules UI, tu n’as plus besoin de définir SERVER_PORT
ni SERVER_HOST
. Et tu pourrais alors remplacer :
main(server, SERVER_HOST, SERVER_PORT);
par :
main(server, app); // `app` est un serveur HTTP Express
Puis dans https://github.com/duniter/duniter-currency-monit/blob/dev-module/lib/webserver.js remplacer (host, port, ...)
par (appParente, ...)
. Les lignes 54-58 n’ont alors plus aucune utilité, tout comme les lignes 63-73 qui seront avantageusement remplacées par :
appParent.use("/duniter-currency-monit", app);
En tant que sous-serveur Express.
La ligne d’injection serait alors :
openNewTab (window.location.origin + '/duniter-currency-monit/currency-monit/', {
Je te balance tout cela de façon désordonnée, mais tu devrais t’y retrouver : le principe est de réutiliser le serveur web de Duniter (sur le port 9220 généralement) plutôt que d’en instancier un nouveau. A noter que tu peux aussi conserver un serveur web que tu ouvres dans le cas de la commande currency-monit [host] [port]
, mais ça te fait gérer 2 cas différents.
Bref, il y a encore un peu de boulot quand même