Open AlixCheval opened 2 years ago
Suggestion : répartir les scan dans des modules par type de dépendance : scan nmap, scan impacket, scan scapy, etc...
En profiter pour corriger la fonction creation_data_nmap(ip_address) -> dict
qui utilise une variable non définie 🥲
De nombreuses fonctions ont des doublons et du code mort.
Par exemple fingerprint_ssh
et scan_snmp_info
semble avoir le même code (ou presque le même)
Edit: cela est dû au fonctionnement de Nmap. Un scan est bel et bien effectué, et son résultat est collecté via nm.all_hosts().all_protocols().items()
De nombreuses fonctions ont des doublons et du code mort. Par exemple
fingerprint_ssh
etscan_snmp_info
semble avoir le même code (ou presque le même)Edit: cela est dû au fonctionnement de Nmap. Un scan est bel et bien effectué.
négatif, ce sont des fonctions totalement différentes, c'est toi qui en a codé la base d'ailleurs et je suis repassé dessus ici : 0ad9307efd02833b511cc17ae3c7526a09b27628
Le début du refactoring a commencé : 1381f4775860107e93c3243f2479a9a0f227d9e7 Pour l'instant, l'architecture du code n'a pas été changée, seuls les doublons ont été enlevés et des docstrings ajoutées. Le code a ainsi été clarifié, mais il reste à découper celui-ci en plusieurs parties afin d'alléger la compréhension du programme.
Le fichier
echosounder.py
a besoin d'être refactorisé (doublons de code, manque d'explication dans certaines fonctionnalités, non respect de la PEP8 notamment).Plusieurs tâches devront être effectuées :
echosounder.py
(suppression des doublons, clarification du code, etc.)