Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

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