Discussion :

[ati00]

Bonjour.

J'ai fais un script simple que je développerais, modifierais par la suite.
Ici il y a que le commencement de base. J'ai un souci concernant les conditions dans ce script.
Que je sois connecté avec le vpn ou non j'ai toujours la même réponse ( ip a changer).

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
 
import os
 
vpn = 1316232503                                  # ip vpn "78.116.29.55" str converti en int = 1316232503   
curl = os.system("curl ipinfo.io/ip")             # int
 
if curl == vpn:
    print("bonne ip")
elif curl != vpn:
    print("ip a changer")
else:
    print("none")

La chaîne vpn est un string avec l'ip "78.116.29.55" , curl un entier.
J'ai alors converti vpn en entier à la place de la chaîne, mais j'ai toujours le même résultat ( ip a changer) en déconnectant le vpn ou en le remettant.

Je remarque qu'en tapant curl sur la console, j'obtiens le code 0 et non l'ip.
Hors je pense que le problème vient de là. Je suis en apprentissage de python autodidacte mais je n'ai pas assez d'expérience pour remédier à ce problème.

j'execute et vpn connecter:
>>>
78.116.29.55
ip a changer

j'execute sans vpn:
>>>
78.116.29.55
ip a changer

>>> curl
0

[danielhagnoul]

Vous ne pouvez pas comparer un int avec le contenu de curl :

Code Python :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
import struct
import socket
import os
 
curl = os.system("curl ipinfo.io/78.116.29.55")
 
print(curl)
 
'''
{
  "ip": "78.116.29.55",
  "hostname": "55.29.116.78.rev.sfr.net",  
  "city": "Douai",
  "region": "Hauts-de-France",
  "country": "FR",
  "loc": "50.3707,3.0792",
  "org": "AS8228 SFR SA",
  "postal": "59351 CEDEX",
  "timezone": "Europe/Paris",
  "readme": "https://ipinfo.io/missingauth"
}0
'''
 
 
def ip2int(addr):
    return struct.unpack("!I", socket.inet_aton(addr))[0]
 
 
def int2ip(addr):
    return socket.inet_ntoa(struct.pack("!I", addr))
 
 
print(int2ip(1316232503))  # 78.116.29.55
print(ip2int('78.116.29.55'))  # 1316232503

J'ai testé avec l'adresse de mon modem, les renseignements fournis par curl ne sont pas fiables.

[wiztricks]

Salut,

Je suis en apprentissage de python autodidacte mais je n'ai pas assez d'expérience pour remédier à ce problème.

Ce n'est pas un problème Python à proprement parler mais de compréhension de ce que fait os.system.

Il se contente de lancer la commande et de retourner un status sous la forme d'un entier entre 0 et 255 qui dira si la commande s'est exécutée ou pas sans problème.
Vous ce que vous voulez récupérer, c'est ce que la commande affiche à l'écran (après avoir testé le status).

Pour çà il est préférable d'utiliser subprocess:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
from subprocess import check_output
curl = check_output("curl ipinfo.io/ip", shell=True)
print(curl)

Et comme curl sera une chaine de caractères (mais des bytes), vous savez à quoi la comparer pour tester.

- W

[Sve@r]

Bonjour

En plus de ton erreur sur le comportement de os.system(), moi je m'interroge sur ton alternative binaire à 3 sorties.
Tu commences par tester si curl est égal à vpn, puis si curl est différent de vpn. Et sinon tu affiches "none".

Et donc c'est quoi pour toi ce 3° état mystérieux où curl ne serait ni égal ni différent de vpn ???

[ati00]

danielhagnoul: Vous ne pouvez pas comparer un int avec le contenu de curl

Sur un navigateur l'adresse http://ipinfo.io/ip donne que l'ip, mais après l'échec du script je me doutais un peu que curl ne donner pas que l'adresse ip, surtout que le test de booléen entre vpn et curl donner False. Merci d'avoir essayer ton script vérificateur.

wiztricks: Ce n'est pas un problème Python à proprement parler mais de compréhension de ce que fait os.system.

C'est que os.system je ne dois pas l’utiliser pour ce cas, je me rend compte que je capture que le code de retour de la commande.
Pour le code avec subprocess et mon code, cela ne fonctionne pas mais merci de ton code, je le retenterais demain ou Mardi.
Mais je vais écarter os.system du script et le remplacer.

Sve@r[: En plus de ton erreur sur le comportement de os.system(), moi je m'interroge sur ton alternative binaire à 3 sorties

Pour l'alternative à 3 sorties else est inutile tu as raison je l'ai mis et cela ne mange pas de pain dans ce test. les conditions c'est pour simplifier le script ici mais après les conditions seront remplacer par une boucle, etc ... pour faire une alternative à Ip Indicator, j’utiliserais notify-send etc.. Mais je comprend pour toi à la base cela peut paraître sans valeur ce test.
Le test des conditions doit fonctionner pour que je puisse ensuite finaliser mon script, parce que le principal c'est si le l'ip du vpn est connecter ou si elle ne l'ai pas.
Bonne soirée.

[danielhagnoul]

Pour interroger https://ipinfo.io/, je vous recommande l'usage du module Python Geocoder : https://geocoder.readthedocs.io/

Pour l'installer : pip install geocoder voir : https://pypi.org/project/geocoder/

Code Python :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
import geocoder
 
g = geocoder.ipinfo("78.116.29.55")
 
print(g.ip)
print(g.json)

[Sve@r]

je l'ai mis et cela ne mange pas de pain dans ce test.

Si, cela mange du pain. Le processeur fera 2 comparaisons là où une suffit

Les conditions c'est pour simplifier le script ici

Simplifier je comprends (une des philosophies Python est justement que la simplicité est préférable à la complexité). Toutefois ta façon de simplifier je comprends moins... print("bonne ip" if curl == vpn else "ip à changer").

Pour le code avec subprocess et mon code, cela ne fonctionne pas mais merci de ton code, je le retenterais demain ou Mardi.
Mais je vais écarter os.system du script et le remplacer.

Tu peux essayer ceci

Code python :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
import subprocess
curl=subprocess.Popen(("curl", "ipinfo.io/ip"), stdin=None, stdout=subprocess.PIPE).stdout.read()

Et pour plus de détails sur subprocess, ses paramètres et la façon de récupérer son résultat, voir ici.

[ati00]

Super!! @ danielhagnoul
je te remercie de me faire connaître geocoder. je vais lire la documentation tout à l'heure, ça à l'air top je le garde à la fin, et je ferai un essai juste après. Je te tiens au courant. Merci


@Sve@r
Lorsque j'avais écris les conditions, je me suis en plus dit " else: none " est inutile de l'ajouter mais je l'ai mis, pour tester quand même. Sur 20 ou 30 fois d'échec sur '==' ou '!=' je suis tombé 1 seul fois sur 'none' et je sais plus comment. lol
Merci de ta nouvelle commande, j'ai essayé mais cela me donne une ip de type Bytes.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
 
import os 
import subprocess
 
curl = subprocess.Popen(("curl", "ipinfo.io/ip"), stdin=None, stdout=subprocess.PIPE).stdout.read() 
 
vpn = "78.116.29.55".encode()      # ip str converti en bytes
 
 
if curl == vpn:
    print("bonne ip")
if curl != vpn:
    print("autre ip")

Bon cela me gène pas au contraire cela me fais apprendre.
Je suis parti alors rechercher sur le net pour convertir un str en bytes, et ici je converti l'ip vpn en bytes. Le problème l'ip de curl en byte à le retour chariot à la fin \n.
Si je la compare à l'ip vpn bytes elles apparaissent semblable mais ne le sont pas pour Python.
La condition ne peut fonctionner pour '=='.

>>> curl
b'78.116.29.55\n'
>>> vpn
b'78.116.29.55'

Bon je repose ma tête chaude aujourd'hui, je ne sais pas si c'est vraiment Python ou bien la chaleur du moi d'Août lol

[wiztricks]

Salut,

Jje ne sais pas si c'est vraiment Python ou bien la chaleur du moi d'Août

Pourquoi écrire:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

vpn = "78.116.29.55".encode()

plutôt que:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

vpn = b"78.116.29.55"

ou encore:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

vpn = b"78.116.29.55\n"

Et si vous ne voulez pas triturer "vpn": curl.decode().strip()

De toutes façons quand vous récupérer la sortie d'un programme, ce seront des chaines de caractères qu'il faudra retravailler... Autant connaitre les méthodes de base pour le faire car peu importe dans ce que vous voudrez coder, il faudra les connaître.

- W

[ati00]

Salut wiztricks.

Pourquoi écrire:
vpn = "78.116.29.55".encode()

Forcément que je ne peux savoir ces commandes, je ne les ai pas étudier, je sais bien qu'il faut étudier avant ( et j'ai appris déjà des bases d'un livre, c'est impossible de tout savoir en tant que débutant et python c'est vaste ) ,et je ne pensais partir sur d'autre voie et en l’occurrence sur la conversion en bytes, donc pour mon problème j'ai voulu le résoudre vite et j'en suis pas loin. J'ai passé la mâtiné à régler mon problème.

[ati00]

Formidable !! cela fonctionne directement avec geocoder.
Je te remercie beaucoup @danielhagnoul c'est ce qui me fallait, en plus il y a des options intéressante pour le futur.
Trop content que ça fonctionne, vpn connecter ou déconnecter la condition fonctionne parfaitement.


Voici le code corriger:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
 
import geocoder
 
g = geocoder.ipinfo('me')
g = g.ip
 
data = "78.116.29.55"      
 
if g == data:
    print("bonne ip")
elif g != data:
    print("autre ip")

En python il y a tant de façon pour arriver au but.
Il me reste maintenant, à modifier mon script en plus sophistiquer mais cela devrait aller.

Mon problème est résolu. J’étudierais subprocess et bytes.
Merci les amis, je met en résolu.

[Sve@r]

En python il y a tant de façon pour arriver au but.

Oui mais tester l'égalité est sinon tester la différence ça fait quand-même partie des pires.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
import geocoder
print("bonne ip" if geocoder.ipinfo('me').ip == "78.116.29.55" else "autre ip")

Sur 20 ou 30 fois d'échec sur '==' ou '!=' je suis tombé 1 seul fois sur 'none' et je sais plus comment. lol

C'était un état quantique intriqué.
Plus sérieusement c'est totalement impossible. Deux choses sont soit égales entre elles, soit non. Il n'y a pas de 3° choix. Je ne dis pas que tu n'as pas eu "none" à l'écran mais il est possible que ton code ait été alors un peu différent et que le "none" ait été mis (par exemple) dans le cas du "else" (une tabulation malencontreuse et une instruction qu'on croyait en dehors d'un bloc se retrouve en fait dans le bloc). Ou autre situation qui fait que "none" s'affiche alors qu'on reste dans l'une des deux seules alternatives possibles.

Mon problème est résolu. J’étudierais subprocess et bytes.

Les bytes sont les chaines encodées en ascii (c'étaient les "str" sous P2). Or le nom "str" a été conservé sous P3 mais fait maintenant référence à l'unicode. Et donc un des soucis du portage P2 => P3 c'est que certaines librairies qui retournaient des str sous P2 ont eu à choisir entre "retourner la même chose" (donc des bytes) ou "retourner ce qui s'appelle pareil" (donc changer l'encoding du truc retourné puisque le même nom fait référence à une chaine encodée différemment) et ont choisi de retourner la même chose sous P3 => donc des bytes
Je vais pas critiquer (je pense que les concepteurs ont eu à faire un choix et ont choisi ce qu'ils estimaient être le moins pire). Il faut juste en être conscient. Perso quand j'utilise une librairie qui me retourne des trucs déjà je regarde ce qu'elle retourne puis j'essaye de convertir tout ce qui entre en unicode (ce qui maintenant se nomme "str"sous P3) le plus tôt possible pour "tenter" d'être le plus serein ensuite dans mon code.

[ati00]

@Sve@r
Oui je suis d'accord avec ta bonne analyse et l'apparition de none cela du arriver comme tu le décrit.
Je confirme ton code est nickel en remédiant avec ma mauvaise conversion bytes corriger par @wiztricks le script fonctionne parfaitement.

Merci de ton commentaire sur Bytes, je ne savais rien de tout ça.
Merci à vous tous et bonne soirée.