OpenEdition / lodel

Science publishing CMS
GNU General Public License v2.0
50 stars 27 forks source link

Pb https #103

Closed ssonnet closed 3 years ago

ssonnet commented 5 years ago

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 commented 5 years ago

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 commented 5 years ago

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

lnprieto commented 5 years ago

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 commented 5 years ago

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

nahuelange commented 4 years ago

Je viens de faire une PR concernant ce problème : https://github.com/OpenEdition/lodel/pull/122