]> git.joonet.de Git - adminer.git/commitdiff
Store several permanent logins
authorJakub Vrana <jakub@vrana.cz>
Fri, 7 May 2010 14:31:14 +0000 (16:31 +0200)
committerJakub Vrana <jakub@vrana.cz>
Fri, 7 May 2010 15:16:09 +0000 (17:16 +0200)
adminer/include/auth.inc.php
adminer/include/functions.inc.php

index 4ffb42cb0e7ec8f6cf33c43da379b592bc1246a2..43382f339186b6c343d144712d1307c6bb8ed6e1 100644 (file)
@@ -12,16 +12,22 @@ if (!$_SESSION["token"]) {
        $_SESSION["token"] = rand(1, 1e6); // defense against cross-site request forgery
 }
 
+$permanent = array();
+if ($_COOKIE["adminer_permanent"]) {
+       foreach (explode(" ", $_COOKIE["adminer_permanent"]) as $val) {
+               list($key) = explode(":", $val);
+               $permanent[$key] = $val;
+       }
+}
+
 if (isset($_POST["server"])) {
        session_regenerate_id(); // defense against session fixation
        $_SESSION["passwords"][$_POST["driver"]][$_POST["server"]][$_POST["username"]] = $_POST["password"];
-       if ($_POST["permanent"] && ($key = $adminer->permanentLogin())) {
-               cookie("adminer_permanent", //! store separately for each driver, server and username to allow several permanent logins
-                       base64_encode($_POST["server"])
-                       . ":" . base64_encode($_POST["username"])
-                       . ":" . base64_encode(encrypt_string($_POST["password"], $key))
-                       . ":" . base64_encode($_POST["driver"])
-               );
+       if ($_POST["permanent"]) {
+               $key = base64_encode($_POST["driver"]) . "-" . base64_encode($_POST["server"]) . "-" . base64_encode($_POST["username"]);
+               $private = $adminer->permanentLogin();
+               $permanent[$key] = "$key:" . base64_encode($private ? encrypt_string($_POST["password"], $private) : "");
+               cookie("adminer_permanent", implode(" ", $permanent));
        }
        if (count($_POST) == ($_POST["permanent"] ? 5 : 4) // 4 - driver, server, username, password
                || DRIVER != $_POST["driver"]
@@ -39,14 +45,20 @@ if (isset($_POST["server"])) {
                foreach (array("passwords", "databases", "history") as $key) {
                        set_session($key, null);
                }
-               cookie("adminer_permanent", "");
+               $key = base64_encode(DRIVER) . "-" . base64_encode(SERVER) . "-" . base64_encode($_GET["username"]);
+               if ($permanent[$key]) {
+                       unset($permanent[$key]);
+                       cookie("adminer_permanent", implode(" ", $permanent));
+               }
                redirect(substr(preg_replace('~(username|db|ns)=[^&]*&~', '', ME), 0, -1), lang('Logout successful.'));
        }
-} elseif ($_COOKIE["adminer_permanent"]) {
-       list($server, $username, $cipher, $driver) = array_map('base64_decode', explode(":", $_COOKIE["adminer_permanent"]));
-       if ($server == SERVER && $username === $_GET["username"] && $driver == DRIVER) {
-               session_regenerate_id(); // defense against session fixation
-               set_session("passwords", decrypt_string($cipher, $adminer->permanentLogin()));
+} elseif ($permanent && !$_SESSION["passwords"]) {
+       session_regenerate_id();
+       $private = $adminer->permanentLogin(); // try to decode even if not set
+       foreach ($permanent as $key => $val) {
+               list(, $cipher) = explode(":", $val);
+               list($driver, $server, $username) = array_map('base64_decode', explode("-", $key));
+               $_SESSION["passwords"][$driver][$server][$username] = decrypt_string($cipher, $private);
        }
 }
 
index 1441d30f42a26a109dc41d6507315b4f1cf3b477..33d0b20afbc4077b53bb584cb299f1a1a7ecc499 100644 (file)
@@ -295,7 +295,7 @@ function auth_url($driver, $server, $username) {
        global $drivers;
        preg_match('~([^?]*)\\??(.*)~', remove_from_uri(implode("|", array_keys($drivers)) . "|username|" . session_name()), $match);
        return "$match[1]?"
-               . (SID ? SID . "&" : "")
+               . (SID && !$_COOKIE ? SID . "&" : "")
                . ($driver != "server" || $server != "" ? urlencode($driver) . "=" . urlencode($server) . "&" : "")
                . "username=" . urlencode($username)
                . ($match[2] ? "&$match[2]" : "")