Project

General

Profile

Actions

Anomalie #1199

closed

caractère manquant dans url redirigée après reconnexion

Added by em zeweb over 6 years ago. Updated over 6 years ago.

Status:
Fermé
Priority:
Normal
Category:
Core
Target version:
Start date:
09/17/2018
Due date:
% Done:

100%

Estimated time:
Version utilisée:

Description

Bonjour,
En revenant sur galette alors que ma session utilisateur a expiré, j'obtiens une page d'erreur 404 Not Found après la reconnexion car le "/" normalement situé dans l'url après "/webroot/index.php" et avant "tableau-de-bord" ou "adherents", par exemple, n'est pas présent.

Merci de votre aide s'il est possible de corriger cette coquille dans le code !

Actions #1

Updated by Christian Hardy over 6 years ago

Je rencontre le même problème mais sur la version 0.9.2-dev.
J'ai le version 0.9.2 installée en production. Pas de problème

Actions #2

Updated by Johan Cwiklinski over 6 years ago

Je n'ai pas le temps de tester toutes les variantes possibles, mais je pense que ce correctif devrait faire l'affaire :

diff --git a/galette/includes/main.inc.php b/galette/includes/main.inc.php
index c5f421ea..2154b022 100644
--- a/galette/includes/main.inc.php
+++ b/galette/includes/main.inc.php
@@ -360,11 +360,15 @@ $baseRedirect = function ($request, $response, $args = []) use ($container) {
  */
 function getGaletteBaseUrl(\Slim\Http\Request $request)
 {
-    return str_replace(
+    $base_url = str_replace(
         ['index.php', $_SERVER['REQUEST_SCHEME'] . '://' . $_SERVER['HTTP_HOST']],
         ['', ''],
         $request->getUri()->getBaseUrl()
     );
+    if (strlen($base_url) && substr($base_url, -1) !== '/') {
+        $base_url .= '/';
+    }
+    return $base_url;
 }

 /**

Actions #3

Updated by Johan Cwiklinski over 6 years ago

  • Category set to Core
  • Status changed from Nouveau to In Progress
  • Assignee set to Johan Cwiklinski
Actions #4

Updated by Johan Cwiklinski over 6 years ago

ping?

Actions #5

Updated by Johan Cwiklinski over 6 years ago

  • Status changed from In Progress to Résolu
  • % Done changed from 0 to 100
Actions #6

Updated by Johan Cwiklinski over 6 years ago

  • Status changed from Résolu to Fermé
Actions

Also available in: Atom PDF