[WIP] Recalbox Manager

[DjLeChuck]

@subs oui tu as bien lu mais je vais voir pour passer ça en Node JS plutôt

@voljega oui tout à fait design responsive mon but sera surtout de l'utiliser sur mon téléphone

[valvolt]

Salut @DjLeChuck
Penses-tu possible d'ajouter une page de login / mot de passe ? Ça manque cruellement à la version présente de recalbox-manager

[DjLeChuck]

@valvolt Je me pose la question de la sécurité d'accès oui, que ce soit sur l'interface que je développe ou sur l'API gérée par @neolao car actuellement n'importe qui peut gérer la recalbox si l'API est activée.

[MikaXII]

J'ai ouvert une issue au sujet de la sécurité https://github.com/recalbox/recalbox-os/issues/1032
Il vaut mieux que l'accès soit géré par l'api et non pas par les différents clients

[DjLeChuck]

Nickel @MikaXII c'est la solution à faire en effet. Reste plus qu'à voir ce qu'en pense @neolao

[Substring]

osef le login. Il ne faut l'activer que dans le cas où le omde secure est activé (cf recalbox.conf)

[Substring]

@DjLeChuck avec @neolao on avait regardé pour un format du recalbox.conf qui serait TRES explicite sur comment gérer les différentes valeurs. Cf https://gist.github.com/substring/e473f24275f25a23cd3ba3c797df74b1

[DjLeChuck]

@subs Comment fonctionne ce mode secure de recalbox ? Une fois activé le password se définit comment ?

Pour la config, oui le format est explicite, la seule interrogation qu'on avait concernait l'utilisation des double quote pour entourer les valeurs ; Au final on voit bien qu'il ne faut PAS les utiliser. Donc une fois que @neolao aura fait la modif sur son API ça sera OK (je l'ai faite sur ma recalbox et pas de problème)

[Substring]

@DjLeChuck tu es sûr que tu as bien regardé le fichier ? Selon les commentaires qui précèdent une variable, tu sais si tu as un bouton on/off, une saisie libre, une dropdown etc ... Seul neolao était au courant de ce truc, il t'a filé le lien ? Sachant que ce n'est pas encore intégré à recalbox officiellement, mais ca peut y venir

pour le mode secure : ca force un mot de passe pour samba au moins et vire le manager aussi on dirait, je ne sais :
https://github.com/recalbox/recalbox-buildroot/commit/c8e5d53b826ebff079ce27f5b7ea49a77eb0c90e
https://github.com/recalbox/recalbox-buildroot/commit/c0554a4f43675c0e13c065ea43dfed7466fc06c4

[DjLeChuck]

@subs Je ne comprends pas ta remarque concernant le fichier. Le format est celui utilisé dans recalbox.conf donc ça me parle oui. Le problème qu'il y a avec l'API de neolao c'est qu'à l'enregistrement d'une donnée string, il l'entoure avec des double quotes. C'est un problème car recalbox croit alors que les double quotes font parti de la valeur alors que non elles servent (normalement) à délimiter la chaîne de caractère (et éviter les problèmes de caractères spéciaux à échapper).

On a bien compris qu'il ne fallait donc pas utiliser les double quotes et il fera la correction (ou je ferais un PR) et c'est tout. Là le fichier que tu me montres va servir à quoi, faire tampon entre le fichier recalbox.conf et l'API ? Je ne sais pas comment est géré l'enregistrement des données je n'ai pas regardé en détail Tout ce que j'ai remonté c'est ce bogue de double quote et le fait que certaines modifications de l'API ne sont pas faîtes instantanément sur le recalbox (comme le volume du son ou la musique de fond par exemple).

Je crois qu'on se mélange sur différentes choses là... ^^'

[Substring]

@DjLeChuck haaaannn t'as pas lu

## EmulationStation menu style 
# Values:
# default: default all options menu
# none: no menu except the game search menu
# bartop: less menu, only needed for bartops
system.es.menu=default

Tu vois la section Values: elle n'existe pas par défaut. C'est un commentaire qui sert à dire à un interpréteur comme t aversion du manager "system.es.menu prend 3 valuers possibles : default, none et bartop, avec les commentaires inhérents". Et il y a un double dièse sur un début de commentaire de variable.

L'idée était d'utiliser le recalboc.conf du share_init puisque c'est lui la référence (mis à jour, non touché par les utilisateurs), et mettre à jour en face le recalbox.conf du share

[DjLeChuck]

@subs Si j'ai lu (seulement le fichier, pas les explications en dessous :P) Mais actuellement ça y ressemble donc pardonne moi de ne pas avoir fait attention à la différence ^^ :

## EmulationStation menu style 
## default -> default all options menu
## none -> no menu except the game search menu
## bartop -> less menu, only needed for bartops
system.es.menu=default

Maintenant que j'ai lu les explications d'en-dessous et que tu me l'a expliqué également je vois de quoi tu parles.

Je ne suis pas encore au fait de comment fonctionnent les différente configs mais du coup là je comprends mieux l'utilisation que tu voudrais voir. En soit ça me va comme ça, puisque de toute façon ça concernera neolao et son API, pas mon manager

[Substring]

@DjLeChuck non neolao se contente d'appliquer des valeurs ou de les extraire. Ces comms ont été ajoutés pour une IHM explicitement, qui permettrait de paramétrer tout le recalbox.conf

[DjLeChuck]

@subs L'API de neolao prend la valeur que tu lui envoies en l'enregistre en l'encapsulant dans des double quotes d'où le bogue dont je parlais : https://github.com/recalbox/recalbox-api/blob/1.1.x/bundle-es6/utils/IniFile.js#L125-L131

Sinon, les commentaire sont très bien oui ça me permet de savoir quoi proposer à l'utilisateur comme options par rapport à ce qui est indiqué sans me poser de questions, donc c'est très bien !

[Substring]

@DjLeChuck justement, au boulot, faut que ta version du manager puisse tout faire sur le recalbox.conf. J'attends ta PR

[DjLeChuck]

@subs Haaa je comprends plus rien... XD Tu ne veux plus que j'utilise l'API et que je fasse directement des modifs dans le recalbox.conf de share_init ?

[Substring]

@DjLeChuck non pas du tout. Tu utilises l'API sans pb, mais ton interface tu la termines pour qu'elle puisse gérer tout recalbox Et surtout :

• le reboot/shutdown
• le script de support
• un bouton "kill running emulator" genre "panic mode" pour éviter que les gens rebootent leur recalbox et couinent qu'elle est cassée
• je ne sais pas quoi encore

NodeJS ou django, à ton aise

[DjLeChuck]

@subs Ha ! Oui ça c'est bon. Je gère déjà le reboot / shutdown (puisque l'API le propose).
Le script de support faudra que je vois comment ça marche.
Pour le kill running emulator et autres je verrais pour faire une PR sur l'API pour ajouter cette fonctionnalité d'abord. Sauf si on part du principe que mon manager s’intégrera dans recalbox directement ce qui me permettra de faire des appels systèmes tranquillement (ou alors qu'une des possibilités de l'API est d'exécuter une commande système et de renvoyer le résultat...)
Mais comme je suis en train de voir pour basculer sur du Node plutôt que du PHP je vais devoir reprendre ce que j'ai déjà...

[Substring]

@DjLeChuck si ton truc est abouti et viable, oui on peut l'intégrer. Le manager dans son état actuel risque fort de prendre la poussière ...

Pour le kill des emulateurs, on a déjà un embryon sur un autre topic en python, mais rien de difficile, si ce n'est que l'utilisation de python est pratique par rapport à configgen (qui a entre autre quasi tous les les binaires des émuls de listés)

[DjLeChuck]

@subs Ok donc raison de plus pour le passer en Node afin de pas s'embêter à avoir PHP encore en plus sur recalbox...

Si tu retrouves le topic je suis preneur dans ce cas, sinon je le chercherais dans tous les cas le moment venu !