Pb https about lodel HOT 5 CLOSED

Re-bonjour,

J'ai trouvé l'origine de mon problème :

header ('Location: http://'. $_SERVER['SERVER_NAME']. ($_SERVER['SERVER_PORT'] ? ':'. $_SERVER['SERVER_PORT'] : ''). C::get('url_retour'));

Cet exemple provient du fichier login.php dans https://github.com/OpenEdition/lodel/tree/master/lodeladmin. Il manque la prise en compte du https. Corrigé comme suit ça passe (je me suis inspiré d'autres appels à la fonction header() dans Lodel) :

header ("Location: http".(C::get('https', 'cfg') ? 's' : '')."://". $_SERVER['SERVER_NAME']. ($_SERVER['SERVER_PORT'] ? ':'. $_SERVER['SERVER_PORT'] : ''). C::get('url_retour'));

J'ai fait une recherche sur la chaine 'Location: http:' dans le code source, il y a 5 occurrences dans 4 fichiers.

Quelqu'un ici m'a conseillé de faire des pull requests pour proposer mes corrections pour intégration dans la branche master. Ont-elles des chances d'aboutir ? Si oui, y-a-t-il des consignes particulières ?

Cordialement.

from lodel.

J'espère que c'est toujours le cas!
@jfriviere une réponse ?

from lodel.

Bonjour
En effet faire des pull requests est un bon moyen pour proposer des corrections à intégrer dans la branche master.
Les modifications proposées ici ont été intégrées dans master.
Merci
Bien cordialement

from lodel.

Si ça a été intégré, cette issue peut-être fermée j'imagine?

from lodel.

Je viens de faire une PR concernant ce problème : #122

from lodel.