M5oul
commented
7 years ago Ton correctif fonctionne mais il assigne la valeur domain.tld/pluxml/{pluxml}.
Ce qui fait que domain.tld/pluxml n’est plus unprotected_uris.
Tu peux le vérifier dans /etc/ssowat/conf.json.
Par contre, ça ouvre un accès sur domain.tld/pluxml/{pluxml}.
Voici ce qui se passe dans le script d’installation :
- L’application est rendu publique pour permettre l’installation avec
curl. - Ensuite, si $is_public=No : cette propriété publique est enlevée.
Et malheur, ça échoue :
+ '[' No == No ']'
+ ynh_app_setting_set pluxml unprotected_uris -d
+ sudo yunohost app setting pluxml unprotected_uris -v -d --quiet
Warning: usage: yunohost app setting [-h] [-d] [-v VALUE] app key
Warning: yunohost app setting: error: argument -v/--value: expected one argumentOk, finalement, je ne connaissait pas l’existance du helper ynh_app_setting_delete.
opi
Lors de l'installation du paquet, en paramétrant l'instance pour qu'elle ne soit pas publique, j'ai rencontré une erreur de syntaxe au niveau de la ligne 94 du script "install" (quelque chose comme argument manquant). Mon souhait étant que l'instance pluxml soit inaccessible publiquement depuis Internet, j'ai modifié le script et procédé à des tests concluants. Je vous soumets cette proposition de correctif pour avis. Ce sont mes premiers pas avec l'intégration d'applications dans YunoHost, tout comme avec Git et Github d'ailleurs, et j'espère ne pas avoir trop dérogé aux règles en vigueur.