Utiliser variable dans une requete curl API cloudflare ne fonctionne pas.

Bonjour à tous,

Voilà j'ai comme projet de faire un script qui mettrait à jour mon IP dans la règles (policies) cloudflare qui autorise les accès à mes service auto-hébergés. Mais le problème n'est pas la j'ai trouvé la bonne commande Curl pour cette mise à jour dans les API cloudflare.
J'ai commencer à faire un script qui détecte les changements d'IP depuis sa dernière exécution (jusque la tout vas bien) et si l'IP a changé je veux exécuté un code qui inclue l'adresse IP stockée dans une variable... et c'est la que les ennuis commencent.
La commande curl fonctionne bien avec une adresse IP mais si à la place de l'adresse IP je tente de mettre la variable ça ne marche plus j'ai essayé pas mal de choses mais ça ne marche pas.
C'est mon premier script j'ai du m'aider de google pour quasiment chaque commande mais la je coince et mes recherches ne donnent pas vraiment de résultat.
Voici mon script l'IP remplacer par une variable et tout à la fin de l'avant dernière ligne "--data" (j'ai bien sur enlevé tout mes tokens perso):

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18

#!/bin/bash new_ip=$(dig +short myip.opendns.com @resolver1.opendns.com) current_ip=`cat current_ip.txt` if [ "$new_ip" = "$current_ip" ] then echo 'pas de changement ip' exit 0 else echo $new_ip> current_ip.txt fi   curl -X PUT "https://api.cloudflare.com/client/v4/accounts/**token de mon compte**/access/apps/**token de l'app**/policies/**token de la regle**" \ -H "X-Auth-Email: moi@mail.com" \ -H "X-Auth-Key: **Mon token d'identification**" \ -H "Content-Type: application/json" \ --data '{"precedence":1,"decision":"bypass","name":"IP","include":[{"ip":{"ip":"$new_ip"}}]}'   echo 'Mise à jour effectuée'

---

Donc j'ai besoin de votre aide pour que la variable "new_ip" soient prise en compte. En l'état la requête part tel quel et cloudflare me renvoie une erreur ("message": "access.api.error.invalid_request: invalid json in policy rule").

Merci d'avance pour votre aide.
Bon dimanche.

Bonjour

Première remarque: si l'iP est inchangée, tu quittes le script. Le "else" n'a donc pas lieu d'être. Et dans les redirections on a l'habitude de coller la redirection au fichier de destination et non pas à la commande => echo "$new_ip" >current_ip.txt.
Ensuite, on écrit toujours les chaines entre quotes => new_ip="$(dig +short myip.opendns.com @resolver1.opendns.com)" et current_ip="`cat current_ip.txt`". Pour une affectation ça n'a pas d'importance mais ça en aura pour les comparaisons donc si on prend l'habitude de le faire tout le temps on ne tombera pas dans le piège idiot où le test [ $toto = $titi ] plantera connement parce que la variable "titi" contient une phrase et que le shell considèrera chacun des mots de la phrase comme autant d'éléments distincts et non pas comme un seul (sans parler de l'autre cas où titi est vide et où le shell ne voyant rien à comparer te répond alors qu'il manque un opérande...)
Concernant la variable "current_ip" on peut se poser la question sur l'utilisation de l'ancienne écriture de substitution de commande `commande` au lieu de la plus récente et plus lisible $(commande) (d'autant plus que tu la connais puisque tu l'utilises juste au dessus) et surtout de l'utilité de cette variable qui n'est utilisée qu'une seule fois.

Pour ton souci principal, il provient du fait que tu utilises des quotes simples pour l'option "--data" et que dans ce cas, une variable n'est alors pas interprétée.
Exemple

Code:

---

1 2 3 4 5

>>> toto="Hello World !!!" >>> echo "$toto" Hello World !!! >>> echo '$toto' $toto

---

D'où le fait que l'utilisation de doubles quotes en tout temps (qui justement ne bloquent ni les variables, ni les substitution de commandes) permet de résoudre beaucoup de soucis liés aux strings avec espace...

Sve@r je te remercie ça n'as pas marché du premier coup mais en m'inspirant d'un autre script j'ai réussi (d’ailleurs ce script utilise bien les doubles quotes je n'avais pas fait attention...)
https://github.com/K0p1-Git/cloudflare-ddns-updater

Pour les bizarreries de mon code. Bah c'est mon premier code et je demandais à google pour chaque ligne "comment mettre un fichier texte dans une variable bash" ou "comment ecrire une variable bash dans un fichier texte" et je prenais le premier résultat qui fonctionnais. J'ai apporté les modifications que tu as suggéré merci.
En revanche pour le else inutile si les IP sont différente il faut bien que je mette else non ?
Et pour les simples quote pareil j'ai bêtement copié le code des documentation clouflare en l'adaptant à mon besoin https://api.cloudflare.com/#access-p...-access-policy

Donc voilà mon script corrigé et fonctionnel:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17

#!/bin/bash new_ip="$(dig +short myip.opendns.com @resolver1.opendns.com)" if [ "$new_ip" = "$(cat current_ip.txt)" ] then echo 'pas de changement ip' exit 0 else echo $new_ip> current_ip.txt fi   curl -X PUT "https://api.cloudflare.com/client/v4/accounts/**token de mon compte**/access/apps/**token de l'app**/policies/**token de la regle**" \ -H "X-Auth-Email: moi@mail.com" \ -H "X-Auth-Key: **Mon token d'identification**" \ -H "Content-Type: application/json" \ --data "{\"precedence\":1,\"decision\":\"bypass\",\"name\":\"IP\",\"include\":[{\"ip\":{\"ip\":\"$new_ip\"}}]}"   echo 'Mise à jour effectuée'

---

Est ce que ce genre de script est assez léger pour être lancé toutes les minutes (même si en vrai je pense plutôt le faire toutes les 10 ou 15 minutes ça me semble suffisant).

En tout cas encore merci pour ton aide je commençais à m'arracher les cheveux :)

Citation:

---

Envoyé par Rutsah

En revanche pour le else inutile si les IP sont différente il faut bien que je mette else non ?

---

Non, puisque dans le cas du "if" le script s'arrête. Corollaire, s'il continue c'est que fatalement tu es dans un "else" implicite. D'ailleurs tu dois le sentir puisque la commande "curl" n'est pas dans le "else" mais ne s'exécute que si les IP sont différentes...

Citation:

---

Envoyé par Rutsah

Donc voilà mon script corrigé et fonctionnel:

---

Les chaines se mettent entre quotes doubles. Toutes les chaines. Seules exceptions, quand on utilise "tr" ou "awk" parce que même si les quotes doubles sont possibles, la syntaxe est plus simple avec des quotes simples (surtout que "awk" utilise le "$" qui pourra foutre le bordel avec le shell si tu ne bloques pas son interprétation).

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

#!/bin/bash new_ip="$(dig +short myip.opendns.com @resolver1.opendns.com)" if [ "$new_ip" = "$(cat current_ip.txt)" ] then echo "pas de changement ip" exit 0 fi   echo "$new_ip" >current_ip.txt curl -X PUT "https://api.cloudflare.com/client/v4/accounts/**token de mon compte**/access/apps/**token de l'app**/policies/**token de la regle**" \ -H "X-Auth-Email: moi@mail.com" \ -H "X-Auth-Key: **Mon token d'identification**" \ -H "Content-Type: application/json" \ --data "{\"precedence\":1,\"decision\":\"bypass\",\"name\":\"IP\",\"include\":[{\"ip\":{\"ip\":\"$new_ip\"}}]}"   echo "Mise à jour effectuée"

---

Citation:

---

Envoyé par Rutsah

Est ce que ce genre de script est assez léger pour être lancé toutes les minutes (même si en vrai je pense plutôt le faire toutes les 10 ou 15 minutes ça me semble suffisant).

---

Il est pas super lourd en tant que tel sauf le "dig" toutes les minutes qui peut être une charge un peu inutile. Si tu penses que 10 ou 15 minutes est un bon compromis alors pourquoi forcer? Tu le mets dans un "cron" et tu ne t'en occupes plus...

Ok, merci pour tout tes bons conseils. Pour la fonction if je ne sais pas pourquoi mais je pensais que le else était obligatoire.
J'étais déjà content simplement que ça fonctionne mais un code nettoyé et optimisé ça ne fait pas de mal, surtout que ça tournera sur un Raspberry (4 mais quand même).

Citation:

---

Code:

---

if [ "$new_ip" = "$(cat current_ip.txt)" ]

---

---

Code:

---

if [ "$new_ip" = "$(<current_ip.txt)" ]

---

Code:

---

if test "$new_ip" = "$(<current_ip.txt)"

---

Citation:

---

Envoyé par N_BaH

if [ "$new_ip" = "$(<current_ip.txt)" ]

---

Comme toujours, tu penses à des trucs auxquels je ne pense pas :applo:

Citation:

---

Envoyé par Sve@r

Code:

---

1 2 3 4 5

#!/bin/bash echo "$new_ip" >current_ip.txt ... curl -X PUT "..." \ --data "{\"precedence\":1,\"decision\":\"bypass\",\"name\":\"IP\",\"include\":[{\"ip\":{\"ip\":\"$new_ip\"}}]}"

---

---

Citation:

---

Envoyé par N_BaH

Code:

---

if [ "$new_ip" = "$(<current_ip.txt)" ]

---

---

Juste une petite remarque à 2 balles en passant:
le fichier "current_ip.txt" contient l'adresse ip courante, suivie d'un caractère de fin de ligne, car on a utilisé echo "$new_ip" et non echo -n "$new_ip" ou echo "$new_ip\c".
Si je fais current_ip="$(cat current_ip.txt)"; echo ${#current_ip}, je constate que la variable ne contient PAS le (ou les) caractère de fin de ligne.

Pourquoi ?

Si je fais :

Code:

---

1 2 3 4

echo -e "coucou\ncoucou">foo.txt od -c foo.txt foo="$(cat foo.txt)" echo ${#foo}

---

j'obtiens 13 caractères, avec le \n intermédiaire, mais pas le final...

Dois-je en conclure que seul le \n final n'est pas récupéré lorsque je mets le résultat du cat dans une variable ?

Citation:

---

Envoyé par jack-ft

Si je fais current_ip="$(cat current_ip.txt)"; echo ${#current_ip}, je constate que la variable ne contient PAS le (ou les) caractère de fin de ligne.

Pourquoi ?

---

Intéressante question :chin:

En effet...

Code:

---

1 2 3 4 5 6 7 8 9

$ echo "coucou" >fic $ wc -c fic 7 fic $ v="$(<fic)" $ echo "[$v]" [coucou] $ echo "[$v]" |od -c 0000000 [ c o u c o u ] \n 0000011

---

L'hypothèse que je peux formuler, c'est que toutes les commandes de récupération de flux suppriment systématiquement le dernier '\n' si présent.

Citation:

---

Envoyé par jack-ft

Dois-je en conclure que seul le \n final n'est pas récupéré lorsque je mets le résultat du cat dans une variable ?

---

Ca me semble logique de le supposer. Mais pas que cat. Ce sera vrai aussi pour ls, pour wc, pour echo même (oui, bien que très con on peut remplacer v="123" par v="$(echo 123)") bref pour toutes les commandes qui affichent des lignes au final. Toutes les lignes seront bien reçues excepté la dernière qui sera privée de son '\n'. Probablement parce que ce '\n' est présumé être ensuite affiché via le echo "[$var]" qui fait généralement suite à la création de "var" ; ou plus simplement parce qu'il est présumé être gênant dans le traitement de l'info renvoyée par ladite commande (en partant du principe que dans la majorité des cas on ne récupère et on ne traite que des infos tenant sur une unique ligne)...

Citation:

---

Envoyé par Sve@r

Intéressante question :chin:

---

N'est-elle pas ?

Citation:

---

En effet...
L'hypothèse que je peux formuler, c'est que toutes les commandes de récupération de flux suppriment systématiquement le dernier '\n' si présent.

---

Même que si y en a plus qu'un, c'est pareil:

Code:

---

1 2 3 4 5 6 7 8 9

$ echo -e "coucou\n\n\n\n\n" >fic $ wc -c fic 12 fic $ v="$(<fic)" $ echo "[$v]" [coucou] $ echo "[$v]" |od -c 0000000 [ c o u c o u ] \n 0000011

---

=> les commandes de récupération de flux suppriment systématiquement les derniers '\n' si présents

Par contre, les \n intermédiaires sont bien conservés.

Bonjour :coucou:

J'avais déjà remarqué que vim ne pouvait pas enregistrer un fichier texte sans ajouter un \n final. Certains vont dire qu'il est malsain de ne pas avoir de \n final. Cela contrevient à la norme POSIX. Certes, d'autres vont dire qu'une norme, ça se respecte, ou pas. Mais si on ne respecte pas. Comment fait-on pour savoir si la dernière ligne d'un fichier est celle terminée par \n, ou la ligne vide suivante qui n'est pas terminée par un \n ? Conclusion : il faut respecter la norme et proscrire les lignes sans \n final.

Citation:

---

Envoyé par Flodelarab

J'avais déjà remarqué que vim ne pouvait pas enregistrer un fichier texte sans ajouter un \n final.

---

Euh si, si le fichier ouvert n'en contient pas à l'origine, il le dit (un message "[noeol]" affiché en bas de l'écran quand on ouvre le fichier) mais il ne le rajoute pas (et je n'ai pas trouvé de moyen de le lui faire mettre donc en général je tape ensuite echo >> fic en faisant super gaffe de bien mettre deux redirections :bug:).

Voici un effet de bord si pas de '\n' sur la dernière ligne:

Code:

---

1 2 3 4 5 6 7

$ unset a $ while read a ; do echo $a ; done < <(echo toto) toto $ unset a $ while read a ; do echo $a ; done < <(echo -n toto) $ echo $a toto

---

donc le read a bien valorisé la variable "a" mais par contre il sort en erreur

Code:

---

1 2 3

$ read a < <(echo -n toto) $ echo $? 1

---

Citation:

---

Envoyé par disedorgue

donc le read a bien valorisé la variable "a" mais par contre il sort en erreur

---

Bien vu :applo:
Cela peut créer des soucis dans les scripts

Code:

---

1 2 3

while read ligne, do traitement "$ligne" done < <(commande)

---

Si la dernière ligne ne contient pas de '\n', elle ne sera pas traitée. En même temps si la dernière ligne ne contient pas de '\n' elle ne correspond alors pas à la norme et étant hors norme, ne peut pas se plaindre d'être hors la loi hors traitement...

Oui mais là, il y a encore une bizarrerie. read pourrait ne pas snober la dernière ligne au seule prétexte de sa fin. awk agit quand même :

Code:

---

1 2 3 4 5 6 7 8 9

$ cat 4cocos coco coco coco coco$ awk -vORS=P '1' 4cocos cocoPcocoPcocoPcocoP$ while read ligne; do echo "pouet";done < 4cocos pouet pouet pouet

---

Il y a bien 4 enregistrements traités avec awk.

Ce n'est pas read qui la snob, car il a bien la valeur de la dernière ligne mais il sort en erreur:

Code:

---

1 2 3 4 5

$ while read a || [ -n "$a" ] ; do echo $a ; done < <(echo -ne "\ntoto\n\ntiti")   toto   titi

---

Citation:

---

Envoyé par Sve@r

Euh si, si le fichier ouvert n'en contient pas à l'origine, il le dit (un message "[noeol]" affiché en bas de l'écran quand on ouvre le fichier) mais il ne le rajoute pas (et je n'ai pas trouvé de moyen de le lui faire mettre donc en général je tape ensuite echo >> fic

---

C'est pourtant pas bien compliqué...

Si tu veux que ton éditeur mette systématiquement un newline quand tu sauves un fichier, il te suffit de customiser la variable require-final-newline et de la mettre à t (ou bien de mettre, à la mano, une ligne comme (setq-default require-final-newline t) dans ton "~/.emacs").
Si tu veux qu'il te pose la question de l'ajouter ou non à la sauvegarde, il suffit de remplacer t par 'ask, par exemple: (setq-default require-final-newline 'ask).

:dehors:

Citation:

---

en général je tape ensuite echo >> fic en faisant super gaffe de bien mettre deux redirections :bug:).

---

C'est aussi pour éviter ce genre de risque que j'ai un set noclobber dans mon "~/.profile"

Citation:

---

Envoyé par jack-ft

Si tu veux que ton éditeur mette systématiquement un newline quand tu sauves un fichier, il te suffit de customiser la variable require-final-newline et de la mettre à t (ou bien de mettre, à la mano, une ligne comme (setq-default require-final-newline t) dans ton "~/.emacs").

---

Mouais. Mais comme 1) j'utilise "gvim" (issu de "vi" mentionné par Flodelarab à qui je répondais) et que 2) emacs ce n'est pas un éditeur...:chin:

Citation:

---

Envoyé par jack-ft

C'est aussi pour éviter ce genre de risque que j'ai un set noclobber dans mon "~/.profile"

---

Ah celle là je ne la connaissais pas :applo:
Allez, adopté (et ça rattrape ta bourde concernant emacs :mrgreen:)

Citation:

---

emacs ce n'est pas un éditeur

---

tout le monde n'est pas d'accord.

Citation:

---

Envoyé par gnu.org

GNU Emacs
An extensible, customizable, free/libre text editor

---