[WIP] Recalbox Manager

[DjLeChuck]

@subs Heuu là je suis pas d'accord... Certes de mon côté le manager doit envoyer une valeur correcte mais je ne suis qu'un client de l'API. Le côté serveur c'est justement l'API donc il FAUT une validation côté serveur ce n'est pas sécurisé sinon et n'importe qui peut envoyer ce qu'il veut comme donnée et réaliser des injections.

Après ça c'est dans la pratique habituelle des choses. Ici comme tu l'as déjà répété le système n'est pas ouvert etc. mais bon tu peux te faire pénétrer ton réseau et voilà...

[Substring]

@DjLeChuck mais c'est déjà le cas ! tu peux envoyer ce que tu veux je parie comme valeur à l'API

[neolao]

J'ai oublié si on peut pour tout type de propriété.

En tout cas, je confirme que c'est à l'API de gérer les bonnes valeurs.
Il pourra même servir les valeurs possibles, ce sont des ressources en GET finalement aussi.

[neolao]

Et potentiellement, comme le frontend a ses traductions pour les noms de propriétés, valeurs et descriptions, l'API peut exposer les traductions aussi.

Un manager web serait comme un frontend à recalbox.

Idéalement, le frontend en sortie vidéo devrait être découplé du backend via une API. Ce qui permettrait d'en changer.
Dans mes ambitions, je voulais allez loin et l'API est la première pierre.

[DjLeChuck]

@subs J'ai pas compris ta réponse ^^' Oui on peut envoyer ce qu'on veut à l'API sans qu'elle ne dise rien et ce n'est pas normal. Elle devrait contrôler les données reçues.

@neolao Pour les valeurs possibles elle pourrait oui, mais ce n'est pas une obligation car c'est à ça que sert une doc aussi, indiquer ce qui est possible de faire ou non, et on en est pas encore là De même pour les traductions elle pourraient être données dans l'API mais ce n'est pas non plus une obligation. L'API peut très bien rester "bas niveau" et ne donner que les valeurs bruts ce n'est pas déconnant.

[neolao]

Bien sûr, je parlais de ressources supplémentaires qu'elle pourrait fournir.

Imagine pour paramétrer la résolution de l'écran qu'une ressource te donne les valeurs possibles par rapport à un diagnostique.

Je vais peut-être faire une différence d'URI pour modifier la configuration et les autres actions. Par exemple, changer le volume, il y a une différence entre modifier la config et modifier le volume courant.
Actuellement, le endpoint est /audio/volume.

Je ferai peut-être un /config/audio.volume pour être dans le schéma /config/{property}.

[Substring]

Je vous explique pourquoi l'API est mal barrée pour faire du contrôle d'intégrité sur ce qu'on lui passe :

• je mets un mode video non supporté par le moniteur, mais formellement correct
• je spécifie un core d'émulateur qui n'existe pas (parce que, faut etre honnête, les cores par console ne seront pas listés dans le recalbox.conf, ils sont déjà dans le es_systems.cfg)
• j'indique un mauvais systeme
• je mets un ratio inexistant (il y en a 3 ou 4 dans la 4.0, plus d'une vingtaine en 4.1)
• il faut balayer les zones libres sans controle (user/pass retroachievements par exemple)
• je spécifie un configfile qui n'existe pas

Bien sûr tous ces cas peuvent être "résolus" mais à quel prix ...

[neolao]

Pour moi, ce sont de mauvaises raisons.
L'API doit avancer au même rythme que les fonctionnalités.

Mais en pratique, ce n'est pas un projet basé sur l'API, ce dernier va avancer avec un autre rythme.
Donc, c'est plus sûr de ne pas contrôler l'intégrité de la valeur.
Par contre, ajouter une ressource qui donne les valeurs possibles, c'est intéressant pour un consommateur de l'API. Au pire, les valeurs ne seront plus bonnes mais il pourra toujours envoyer ce qu'il veut.

[Substring]

Je peux te donner la réponse à chaque question que je t'ai posée au-dessus quasiment. Mais toutes les réponses ne seront pas dans le recalbox.conf, ou talors ca veut dire reporter des valeurs d'autres fichiers dans le recalbox.conf. C'est suicidaire pour la maintenance.

Au final, tu te rabats bien sur "l'API fera son boulot quelle que soit la valeur qu'on lui passe" puisque tu reportes les valeurs un cran au-dessus, et donc l'IHM

[neolao]

Tout n'a pas besoin d'être dans recalbox.conf évidemment.
On peut lire d'autres fichiers bien sûr.

Il est possible tout de même de contrôler au moins l'intégrité des propriétés :

• booléenne
• nombre dans un interval
• choix dans une liste finie (et donc rajouter les nouvelles valeurs quand il faut)

Le reste est un développement supplémentaire qui pourrait dépendre d'autres fichiers comme es_systems.cfg. Et donc si ça évolue, l'API s'adaptera automatiquement.

Le mode video vient de la réponse à tvservice.

Bref, on peut tendre vers ça petit à petit. L'intégrité n'a pas besoin d'être absolue.

[DjLeChuck]

Mon pauvre sujet devient n'importe quoi Il n'y a pas un endroit où on pourrait débattre de tout ça qui soit fait pour (je demande hein, m'en fiche que mon topic serve à ça :P) ?

[acris]

Salut
on peut splitter le sujet , un modérateur qui suit le débat le fera

[DjLeChuck]

Hello,

Des nouvelles du front.
Je n'ai pas du tout avancé sur la partie visible de l'iceberg car je cherchais une autre techno que PHP afin de rendre l'outils utilisable plus facilement pour tous.

Puis, @subs m'a dit qu'il pourrait finir directement sur recalbox afin de remplacer l'existant alors je me suis dit que j'allais partir sur un serveur nodejs, puis après avoir testé j'ai pas aimé, donc je me suis dis pourquoi un serveur en Go, puis j'ai testé et j'ai pas aimé, donc je pars au final sur du Django comme c'est le cas pour le manager actuel comme ça je sais que ça tournera sans problème.
Du coup, je dois en revenir là où je m'étais arrêté avec la version PHP afin de pouvoir vous montrer plus de choses...

[voljega]

@DjLeChuck erf c'est con parce que si c'était en JS j'aurais pu aidé un peu

[neolao]

Sinon en gardant un node qui sert une page, vous faites une Single Page App en React.

[DjLeChuck]

@voljega @neolao merci mais je pars bien avec Django au final

J'en suis revenu là où j'en étais donc je vais pouvoir continuer avec d'autres page que audio et config système

[neolao]

En servant juste des fichiers statiques, ton empreinte mémoire sera plus bas pour la machine.

[ouaich]

Hello,

Bravo pour ce projet, en parcourant rapidement les possibilités de l'API (même si je ne comprend pas tout ) je vois qu'il y a plein de possibilités et du coup autant de réglages qu'on aura plus a faire en ligne de commande ou en modifiant des fichiers de config.

J'imagine déjà, quand l'API évoluera, un 2ème petit écran avec en fonction du système choisi une image avec le mappage des touches ou un joli screenshot du jeu en cours ...

J'en salive d'avance, et bon courage.

[Substring]

@DjLeChuck ca va ? On n'a plus de news

[DjLeChuck]

@subs Pfiouuuu ouais ça va, c'est la galère en moment niveau temps dispo pour avancer sur le manager. Je m'y remets ce soir, coûte que coûte, surtout que je n'avance pas bcp à force de changer de techno...

@ouaich Merci pour les encouragements