get Variablen gehen bei weiterleitung mit header(location:) verloren

NyctalusNoctula

Hallo

Ich bin grad an ner größeren Website dran und hab ein Problem.
die User sollen sich auf jeder Seite und zu jeder Zeit ohne Problem ein- und wieder ausloggen können.

Wenn nun aber jemand ein Such-Formular halb ausfüllt, sich dann einloggt, wird er ja zur login-Seite geleitet und von dort mit

PHP

$lastsite = parse_url($_SERVER['HTTP_REFERER'], PHP_URL_PATH);
header("Location: $lastsite");

wieder zurück zur Ursprungsseite.

Leider gehen dabei eben die get Variablen der Suche verloren und das Formular ist wieder leer.

Nachtrag: Noch nicht abgeschickte Variablen können von mir aus sogar verloren gehen. Aber bereits gesetzte Parameter, die auch schon in der URL mit drin stehen sollten auf jeden Fall bleiben.

Kennt ihr eine Lösung für dieses Problem?
Habe schon diverse Servervariablen durchgreschaut und nichts passendes gefunden.

Afrael

Warum übergibst du nicht $_SERVER['REQUEST_URI'] an das login-Script und leitest dann dorthin zurück?

Ace <3st

Wir machen das wie folgt-
Du hast bestimmt dein loginbereich links oder rechts.
Dann setzt du das an die stelle:

PHP

<?php
if($_COOKIE['loggedin'] == "yes")
{
(include("intern.php"));
}
else
{
?>


  
<form action="login.php" method="POST"> 
    <table> 
         <tr> 
            <td><b>Name:</b></td> 
            <td><input type="text" name="logname"></td> 
        </tr> 
        <tr> 
            <td><b>Pass:</b></td> 
            <td><input type="password" name="logpw"></td> 
        </tr> 
        <tr> 
            <td></td> 
            <td><input type="submit" name="sendit" value="Login"></td> 
        </tr> 
    </table> 
</form> 
<?php
    }
?>

Alles anzeigen

In der Login.php sollte dann

PHP

setcookie("loggedin", "yes", time()+60*60*60*60);

erscheinen, damit er das auf YA setzt!
Unsere Login.php sieht zb so aus:

PHP

<?php 


Connect zensiert***


$sql = "SELECT ". 
    "Id, Nickname, Kennwort ". 
  "FROM ". 
    "benutzerdaten ". 
  "WHERE ". 
    "(Nickname like '".$_POST['logname']."') AND ". 
    "(Kennwort = '".md5 ($_POST['logpw'])."')"; 
$result = mysql_query ($sql); 


if (mysql_num_rows ($result) > 0) 
{ 
while($row = mysql_fetch_object($result))
{
setcookie("loggedin", "yes", time()+60*60*60*60);
setcookie("id", $row->Id, time()+60*60*60*60);
setcookie("paw", $row->Kennwort, time()+60*60*60*60);
setcookie("name", $row->Nickname, time()+60*60*60*60);
}
header ("Location: index.php"); 
} 
else 
{ 
  header ("Location: index.php"); 
} 
?>

Alles anzeigen

Ist nicht das sicherste aber vorerst übernehmbar.

Die Intern.php kannst du dir selbst gestalten

Logout.php

PHP

<?php
$in = setcookie("loggedin","",time()-3600); 
$id = setcookie("id","",time()-3600); 
$paw = setcookie("paw","",time()-3600); 
$name = setcookie("name","",time()-3600); 


if($in AND $id AND $paw AND $name == true)
{
header ("Location: /forum/index.php"); 
}
?>

Alles anzeigen

wir rufem unsere urls mit folgender $_GET['']; funktion ab:

PHP

<form action="forum_creat_thread.php?n=<?php echo $_GET['n']; ?>" method="post">

Den link dazu verlinkst du so:

PHP

<?php
connect ****


$sql = "SELECT * FROM welchetabelle";
$update = mysql_query($sql);
while($row = mysql_fetch_object($update))
    {
        echo "<a href=\"/clan/php/new/show_forum.php?n=$row->threadname\">".$row->threadname."</a>"; 
    }
?>

Alles anzeigen

Pion

Nein das ist nicht nur ein bischen unsicher das ist absoluter unsicher;)

Also mach doch das:

header("Location: $lastsite?deine alte url dranhängern");

Oder du erstellst $_SESSION in der du eben die Aktion vor dem Header speicherst

NyctalusNoctula

Afrael: REQUEST_URI funktioniert irgendwie gar nicht. Da leitet er mich immer zum login weiter. Ist also sozusagen dann ne Dauerschleife. Ich will ja wissen, auf welcher Seite der Benutzer war bevor er sich eingeloggt hat.

@Ace_<3st: du hast mein Problem glaube ich nicht richtig verstanden. Das Problem liegt in der Weiterleitung nach dem Login auf die Seite, auf der der Nutzer vorher war. Bei mir kann er soch von zig Seiten aus einloggen und muss dann auch dahin zurück. In deinem Code wird ja immer zur index.php geleitet. Den Login mach ich natürlich mit sessions.

@Pion: daran meine alte url dranzuhängen hab ich auch schon gedacht. Aber dazu müsste ich mit zig if-Schleifen alle Möglichkeiten von übergebenen Variablen druchgehen, oder nicht?
Ne Session hab ich ja soweiso für jeden Nutzer. Kannst du mir näher erklären, wie du die get Variablen da reinspeichern willst? Gibts da ne elegante Möglichkeit zu?

phore

Wie wärs mit

PHP

<?php
  // vor dem login
  session_start();  
  $_SESSION['get_vars'] = $_GET;

  // nach dem login - url wieder aufbauen
  $url = '';
  $i = 1;  
  foreach ($_SESSION['get_vars'] as $key => $value)
    {
    $separator = ($i == 1) ? '?' : '&';          
    $url.= $separator.$key.'='.$value;
    $i++;
    }

  header('Location: '.$_SERVER['SERVER_NAME'].$url);
?>

Alles anzeigen

Gruss

NyctalusNoctula

so, ich hab die Lösung.

Mann bin ich doof. Manchmal sieht man wohl den Wald vor lauter Bäumen nicht mehr.

beim HTTP_REFERER kommen die Get-Variablen natürlich doch mit. Nur habe ich diese mit parse_url abgeschnitten, da ich auch noch auf die Richtigkeit des Hosts prüfe.

Mein code sieht nun so aus:

PHP

//HTTP Referer auslesen und zur vorigen seite zurückschicken, falls diese auf dem Server liegt
if (!empty($_SERVER['HTTP_REFERER'])) {
    $host = parse_url($_SERVER['HTTP_REFERER'], PHP_URL_HOST);
    if ($host == 'www.mein-server.de') {
        $lastsite = parse_url($_SERVER['HTTP_REFERER'], PHP_URL_PATH);
        $query = parse_url($_SERVER['HTTP_REFERER'], PHP_URL_QUERY);
        if($query != ''){
            $lastsite .= '?' . $query;
        }
    }
    else {
        $lastsite = '/';
    }
}
else {
     $lastsite = '/';
}
header("Location: $lastsite");
exit();

Alles anzeigen

Bandit

Warum ist es schlecht, mit dem Referer zu arbeiten?
http://www.php-faq.de/q-http-referer.html

NyctalusNoctula

Zitat von bandit600

Warum ist es schlecht, mit dem Referer zu arbeiten?
http://www.php-faq.de/q-http-referer.html

Das ist mir durchaus bewusst. Bessere Varianten?
außerdem steht ja in deinem Link auch nur, dass man in Sicherheitskritischen Fragen davon Abstand nehmen sollte.
Wenn der User den Referer deaktiviert hatm, wird er halt immer zur index.php weitergeleitet. Dann muss er eben damit leben.

The User

Session nehmen?
Per ?' . $_SERVER['HTTP_QUERY_STRING'] dranhängen?

thilda

Man könnte doch auch einfach alles in ein input-Feld stecken und das mit 'hidden' markieren oder?

Tags