Skip to content

fix: débogage API Python + client Node - Mohamed#13

Open
ororck wants to merge 2 commits into
mainfrom
fix/mohamed-debug-python-node
Open

fix: débogage API Python + client Node - Mohamed#13
ororck wants to merge 2 commits into
mainfrom
fix/mohamed-debug-python-node

Conversation

@ororck

@ororck ororck commented May 21, 2026

Copy link
Copy Markdown
Collaborator

2. Bugs identifiés

Projet Python — python-api/ :

# Fichier Ligne Type d'erreur Description du problème
1 requirements.txt 2 Nom de paquet incorrect flaskk n'existe pas sur PyPI
2 app.py 18 SyntaxError : manquant à la fin de def parse_logs(filepath)
3 app.py ~30 NameError Variable errors utilisée à la place de erreurs dans parse_logs
4 app.py 50 NameError Variable log_file non définie utilisée à la place de "server.log"
5 config.json Mauvaise configuration Port 50001 au lieu de 5000

Projet Node.js — node-client/ :

# Fichier Ligne Type d'erreur Description du problème
1 package.json Nom de paquet incorrect axioss n'existe pas sur npm
2 app.js 12 MODULE_NOT_FOUND require('axioss') — même faute de frappe que dans package.json
3 app.js 21 Mauvaise propriété axios response.body n'existe pas dans axios, la bonne propriété est response.data

3. Corrections apportées

  • Bug 1 : requirements.txt — corrigé flaskk==3.0.0 en flask==3.0.0 car le nom officiel du paquet sur PyPI est flask.
  • Bug 2 : app.py ligne 18 — ajouté : en fin de def parse_logs(filepath): car toute définition de fonction Python requiert ça.
  • Bug 3 : app.py — corrigé errors en erreurs pour correspondre au nom de variable déclaré en début de fonction.
  • Bug 4 : app.py ligne 50 — remplacé la variable indéfinie log_file par la chaîne "server.log", nom réel du fichier de logs dans le dossier.
  • Bug 5 : config.json — corrigé le port 50001 en 5000, valeur attendue par l'architecture du projet et par le client Node.
  • Bug 6 : package.json — corrigé "axioss" en "axios" car c'est le nom exact du paquet HTTP sur npm.
  • Bug 7 : app.js ligne 12 — corrigé require('axioss') en require('axios') pour correspondre au paquet installé.
  • Bug 8 : app.js ligne 21 — remplacé response.body par response.data car dans axios, les données de la réponse serveur sont accessibles via .data (cf. documentation officielle axios-http.com/docs/res_schema).

@ororck ororck requested a review from alderichoarau May 21, 2026 14:25
@ororck ororck added the bug Something isn't working label May 21, 2026
Comment thread node-client/app.js Outdated

const axios = require('axios');
// BUG 6 — Le nom du module importé ici est incorrect
const axioss = require('axios');

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pourquoi garder le nom de variable erroné ?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pour aller au plus simple, mauvaise habitude dsl c'est toi le goat je corrige ça

Comment thread python-api/app.py Outdated
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")

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pourquoi ne pas aller chercher ça depuis le fichier de config au lieu de l'hardcoder ?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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"])

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants