Skip to content

Ajout d'une propriété info pour obtenir l'IP courante à partir de l'@ MAC en mode ARP#22

Open
lerouxi wants to merge 2 commits intoNextDom:masterfrom
lerouxi:master
Open

Ajout d'une propriété info pour obtenir l'IP courante à partir de l'@ MAC en mode ARP#22
lerouxi wants to merge 2 commits intoNextDom:masterfrom
lerouxi:master

Conversation

@lerouxi
Copy link

@lerouxi lerouxi commented Sep 29, 2020

Bonjour,

Suggestion pratique qui permet d'obtenir l'adresse IP à partir d'une adresse MAC en mode ARP. c'est utile pour tous les
plugin basés sur des adresses IP dans un réseau en adressage dynamique, ou dans un réseau limité à 10 adresses IP fixes (SFR BOX)...

c'est aussi très pratique pour identifier les périphériques à partis de leur IP...

3 cas :

  • Jamais détecté : "!.!"
  • IP courante : "192.168.xxx.xxx"
  • Dernière IP utilisée : "!192.168.xxx.xxx!"

Exemple avec la widget info :

image

J'ai en même temps fait un peu de formatage (automatique sur vscode), 2 ou 3 améliorations graphiques et adapté la commande ARP pour la rendre plus rapide et plus efficace sur des réseaux mixtes (switch, lan, wifi)

- Formatting sources
- Fix Orthographe fr
- Fix "My Pings" add command GUI
- Add info field  for Current IP reverse looking from MAC adress in ARP mode
- Tune ARP request
Fix Ping subtype (Last commit fix)
@Gis70
Copy link

Gis70 commented Sep 29, 2020

Salut merci pour le taf, on va regarder pour intégrer tes modifs après test.

@slobberbone
Copy link

Yes merci @lerouxi ! On test et on validera ou te fera un retour si nécessaire ;)

@lerouxi
Copy link
Author

lerouxi commented Sep 29, 2020

Ok, désolé pour le formatage du code, je l'ai activé en autosave sur vscode
J'utilise cette version modifiée du plug-in depuis quelques mois en mode arp, par contre je n'ai pas trop testé les autres modes.

Surtout ne pas hésiter pour les commentaires ou questions 😉

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants