Sikkert login system

Ja og nej. Du gemmer brugernes kodeord som klartekst i databasen. Hvis du har andre huller andre steder risikerer du at kompromittere brugernes kodeord. Gem dem i databasen som en md5-hash. Mener du ikke selv at det er nødvendigt er det helt unødvendigt for dig at bruge md5 overhovedet. Du kan blot checke om kodeordene matcher.
Andre kommentarer: Du sætter status til "Online", men hvis brugeren bare lukker browseren er han stadig online indtil han har logget på igen, og logget ud. Et alternativ er at sætte et felt med et tidsstempel, f.eks. "last_activity", og sætte den hver gang brugeren laver noget, og så definere at en bruger er online hvis last_activity er inden for de sidste 5 minutter f.eks.
str_replace er unødvendig da du bruger mysql_real_escape_string...
tjekbruger og strtolower er unødvendig, du henter where brugernavn='$usernr2'
Her er en lidt clean kode ud fra det du har skrevet:

<?

session_start();
include "connect.php";

$cookuser = $_SESSION['user'];
$cookpass = $_SESSION['pass'];
$cookid = $_SESSION['id'];

if(isset( $_GET['side'] ) && $_GET['side'] == 'nightlife')
{
    $siden = "?side=nightlife";
}
if(isset($cookuser))
{
    header ("location:http://localhost/login/loggetind.php$siden");
}
else
{
    $pass = $_POST["kodeord"];
    $user = trim( $_POST["brugernavn"] );
    $user = mysql_real_escape_string($user);
    if(!empty($user))
    {
        $db = mysql_query("SELECT * FROM login WHERE brugernavn='$user'");
        if ( mysql_num_rows( $db ) > 0 )
        {
            $vis = mysql_fetch_array($db);
            if($vis['kodeord'] == $pass)
            {
                if($vis["accepteret"] == 'ja')
                {
                    $_SESSION['user'] = $usernr2;
                    $_SESSION['pass'] = $pass;
                    $_SESSION['id'] = $vis['id'];
                    mysql_query("UPDATE login SET status = 'Online' WHERE id=$vis[id]");
                    $denne_side = "Login > $_SESSION[user] er logget ind";
                    include "bevaeg.php";
?>
<script type="text/javascript">
top.location.href = "http://localhost/index_ny.php";</script><?

                }
                else
                {
                    $ikke_aktiv = "Din bruger er<br>ikke aktiveret";
                    $denne_side = "Login > $usernr2 forsøger at logge ind, men brugeren er ikke aktiveret";
                    include "bevaeg.php";
                }
            }
            else
            {
                $forkert = "Forkert brugernavn<br>eller kodeord";
                $denne_side = "Login > $user forsøger at logge ind med $pass";
                include "bevaeg.php";
            }
        }
    }
}

?>

Tak for de gode råd :)
Kan du hurtigt forklare eller ved du hvor der er nogle gode artikler om "tidsstempel"?

Idéen er blot, at du ved på hver side, checker om brugeren er logget ind (vha din session)- hvis han er, så laver du en hurtig query:

$sql = "UPDATE user SET lastactivity = " . time() . " WHERE userid = " . $_SESSION['id'];

Så definerer du bare, at en bruger er online, hvis han har genereret aktivitet inden for de sidste 5 minutter f.eks.
For at lave en liste over online brugere kan så:

$sql = "SELECT * FROM user WHERE lastactivity >= " . ( time() - 300 );

300 er i sekunder, altså 5 minutter.

OK :) mange tak for det gode eksempel
det prøver jeg at arbejde lidt videre med.
smid et svar d;

ok :) men det sidste jeg skrev var nu et svar :D

doh ;D
Det var mig der var lidt langsom der :D
15 point til coderdk