[Script] horoscope
#31
Dans un monde idéal où tous les sites https utilisent TLS 1, ou même sur un Eggdrop avec seulement ce script chargé, tout fonctionnerait parfaitement :p

Deux règles importantes :
- ne pas register le protocole https depuis le namespace eval, car il n'est lu qu'une fois au chargement du script et suppose qu'aucun autre script ne tentera de le register à nouveau avec des paramètres différents; il faut le faire au début de la procédure contenant http::geturl
- toujours unregister le protocole https (::http::unregister https) après usage; penser aussi à le faire avant chaque return interrompant le déroulement de la proc contenant http::geturl

CrazyCat a écrit :Merci pour l'info. Comme je ne fais jamais d'erreurs, (...) Very Happy
Répondre Avertir
#32
Merci MenzAgitat, pour la solution apportée.
Petit détail concernant les signes astrologiques, dans la variable prévue à cet effet, il est indiqué "poissons" et non "poisson".

EDIT : il semblerait que le site l'ait appelé "poissons" mais ont également prévu la version au singulier, "poisson", ce qui ne risque donc pas de casser le script si modification de la variable signs.
Répondre Avertir
#33
(07/06/2017, 14:59)MenzAgitat a écrit : Deux règles importantes :
- ne pas register le protocole https depuis le namespace eval, car il n'est lu qu'une fois au chargement du script et suppose qu'aucun autre script ne tentera de le register à nouveau avec des paramètres différents; il faut le faire au début de la procédure contenant http::geturl
- toujours unregister le protocole https (::http::unregister https) après usage; penser aussi à le faire avant chaque return interrompant le déroulement de la proc contenant http::geturl
C'est pas faux, je vais corriger le script en ce sens.

(07/06/2017, 15:31)Destiny a écrit : EDIT : il semblerait que le site l'ait appelé "poissons" mais ont également prévu la version au singulier, "poisson", ce qui ne risque donc pas de casser le script si modification de la variable signs.
J'ajouterai aussi cette correction dans la version suivante, par contre ça ne sera pas avant la fin de semaine.
Répondre
#34
Finalement, j'ai trouvé quelques minutes pour corriger le script.
changelog a écrit :Plusieurs modifications:
- (dés)activation via le flag +/-horo en party-line (.chanset #canal +/-horo)
- correction de l'intialisation du https (merci MenzAgitat)
- le script accepte désormais "poisson" et "poissons"
- petite correction d'encodage pour les accents
Répondre
#35
Il manque un ::http::cleanup dans la proc horoget : actuellement, le token n'est nettoyé qu'en cas d'erreur.

---------------------------------------------

(03/06/2017, 17:46)CrazyCat a écrit : Note importante : il faut la version 1.6.5 du package tls, la version 1.6 ne fonctionne pas, le site étant en tls1.2.
-->package require tls 1.6.5


Désolé de te redonner du boulot Very Happy
Répondre Avertir
#36
Pas de souci, il vaut mieux me donner du boulot comme ça plutôt que de me faire chercher un bug que je ne reproduis pas Wink
Répondre
#37
Voila, version 0.4.1 en ligne Smile
Répondre
#38
Bonjour j'ai testé le scripts récemment mais il me dit qu'il ne trouve pas les fichiers dans la db, est-ce qu'il faut les créer à la main ?

Cordialement
Répondre Avertir
#39
Les fichiers, non, mais le répertoire databases oui.
Répondre
#40
J'ai bien créer le dossier databases dans scripts mais pourtant rien ne se passe et je n'ai pas non plus de message d'erreur en PL.


Citation :<T3nS> !horo lion

<T3nS> !horoscope lion
Répondre Avertir
#41
Pas dans scripts, dans le répertoire où se trouve l'eggdrop. Le répertoire databases doit être à la même hauteur que le répertoire scripts.
Répondre
#42
C'est fait, j'ai rechargé le scripts je l'ai remis sur entretien mais toujours le même résultat il ne se passe rien et pas de message d'erreur en PL :'(
Au temps pour moi il donne les réponses cette fois, ce serait bien qu'il le fasse en notice, je vais voir si c'est configurable.

Merci
Ce n'est pas configurable mais j'ai réussi à modifier le code source pour ça.
Problème résolu.

Merci CrazyCat tu es le meilleur.
Répondre Avertir
#43
(29/10/2017, 17:17)T3nS a écrit : Au temps pour moi il donne les réponses cette fois, ce serait bien qu'il le fasse en notice, je vais voir si c'est configurable.

Ce n'est pas configurable mais j'ai réussi à modifier le code source pour ça.
Je vais ajouter cette option, c'est vrai que ça peut être utile.
Répondre
#44
Oui tu peux rajouter l'option mais c'est vrai que comme l'horoscope est très complet ça risque de flood plus qu'autre chose. j'ai testé en notice mais je me suis vite ravisé en le remettant en privé ^^.

Cordialement.
Répondre Avertir


Atteindre :


Utilisateur(s) parcourant ce sujet : 1 visiteur(s)
Tchat 100% gratuit -Discutez en toute liberté