Pb https

[ssonnet]

Bonjour,

Nous sommes en train de mettre en place Lodel pour la pré-production et nous rencontrons un problème avec le https. J'ai pu faire l'installation en mode sécurisé, par contre après installation, lors de mes tentatives de connexion/déconnexion à l'admin, je tombais sur une page blanche avec l'erreur suivante :

Bad Request

Your browser sent a request that this server could not understand.
Reason: You're speaking plain HTTP to an SSL-enabled server port.
Instead use the HTTPS scheme to access this URL, please.


Le "https" était enlevé de l'url et apparaissait à la place un ":443" à la fin de l'url...

Nous avons désactivé le https mais souhaiterions pouvoir le mettre en place : avez-vous des conseils ? La mise en place du https chez OpenEdition a-t-elle nécessité des adaptations du code de Lodel ?

Cordialement.

[ssonnet]

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.

[nahuelange]

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

[lnprieto]

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

[nahuelange]

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

[nahuelange]

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