fix: débogage API Python + client Node - Mohamed#13
Open
ororck wants to merge 2 commits into
Open
Conversation
|
|
||
| const axios = require('axios'); | ||
| // BUG 6 — Le nom du module importé ici est incorrect | ||
| const axioss = require('axios'); |
Collaborator
There was a problem hiding this comment.
Pourquoi garder le nom de variable erroné ?
Collaborator
Author
There was a problem hiding this comment.
pour aller au plus simple, mauvaise habitude dsl c'est toi le goat je corrige ça
| result = parse_logs(config["api"]["log_file"]) | ||
| # BUG 4 — La variable passée en argument n'est définie nulle part | ||
| # Quel fichier de logs doit-on analyser ? server.log | ||
| result = parse_logs("server.log") |
Collaborator
There was a problem hiding this comment.
Pourquoi ne pas aller chercher ça depuis le fichier de config au lieu de l'hardcoder ?
Collaborator
Author
There was a problem hiding this comment.
j'ai le droit de dire que c'est parce que je suis malade ?
non meme en pleine forme j'aurais fait pareil, c'est tellement mieux d'ecrire
"server.log" au lieu de : result = parse_logs(config["api"]["log_file"])
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
2. Bugs identifiés
Projet Python —
python-api/:requirements.txtflaskkn'existe pas sur PyPIapp.py:manquant à la fin dedef parse_logs(filepath)app.pyerrorsutilisée à la place deerreursdansparse_logsapp.pylog_filenon définie utilisée à la place de"server.log"config.json50001au lieu de5000Projet Node.js —
node-client/:package.jsonaxiossn'existe pas sur npmapp.jsrequire('axioss')— même faute de frappe que dans package.jsonapp.jsresponse.bodyn'existe pas dans axios, la bonne propriété estresponse.data3. Corrections apportées
requirements.txt— corrigéflaskk==3.0.0enflask==3.0.0car le nom officiel du paquet sur PyPI estflask.app.pyligne 18 — ajouté:en fin dedef parse_logs(filepath):car toute définition de fonction Python requiert ça.app.py— corrigéerrorsenerreurspour correspondre au nom de variable déclaré en début de fonction.app.pyligne 50 — remplacé la variable indéfinielog_filepar la chaîne"server.log", nom réel du fichier de logs dans le dossier.config.json— corrigé le port50001en5000, valeur attendue par l'architecture du projet et par le client Node.package.json— corrigé"axioss"en"axios"car c'est le nom exact du paquet HTTP sur npm.app.jsligne 12 — corrigérequire('axioss')enrequire('axios')pour correspondre au paquet installé.app.jsligne 21 — remplacéresponse.bodyparresponse.datacar dans axios, les données de la réponse serveur sont accessibles via.data(cf. documentation officielle axios-http.com/docs/res_schema).