Login Script

<?php
if (Check_login()) {
    print "<center><h2>Dette er side 1</h2></center>";
} else {
    include("./error.html");
    /* Vi kan også sende folk videre vha. header('location: error.html') */
}
?>

Nej det virker desværre ikke

dit problem er , som dragothica viste, at du har glemet et sæt parenteser her:

if Check_login() {

det skal nemlig være

if(Check_login()) {

Hvad du ellers får af syntax-fejl kan vi jo ikke vide.

Derudover så er

$_SESSION["ip"] == $_SERVER["REMOTE_ADDR"]

noget værre fis. Jeg vil anbefale at du lærer at bruge sessions før du fortsætter, www.php.net/session

Hmmmm
Hvis jeg laver den ændring I foreslår får jeg flg. error
Fatal error: Call to undefined function check_login() in C:\apache\htdocs\Havn\Minefiler\side1.php on line 2

Er fejlen ikke noget med at Side1.php ikke ved hvor den skal kigge efter functionen check_login().


Jeg burde måske have fortalt lidt mere om hvor dan jeg ønsker af anvende login'et

på min indexside logger man ind, hvorefter man skulle kunne vælge sider fra en menu, siderne skal så kunne checke om en bruger er logget in



settings.php
<?php
$base_dir = "minefiler/";

$base_file = "bruger.php";

$users = array("admin" => "adminpassword", "showsource" => "viskode", "phpbegynder" => "phpnewbie");

$defalut_file = $base_dir ."bruger.php";

function check_login() {

global $users;

    if(isset($_SESSION["user"], $_SESSION["ip"]) && isset($users[$_SESSION["user"]]) && $_SESSION["ip"] == $_SERVER["REMOTE_ADDR"]) {
    return true;
    }
return false;
}
session_start();
if(isset($_POST["logud"])) {
    $_SESSION = array();
    session_destroy();
}
if(isset($_POST["brugernavn"], $_POST["password"])) {
$default_text = "Forkert bruger og/eller password!";
    if(isset($users[$_POST["brugernavn"]])) {
        if($users[$_POST["brugernavn"]] == $_POST["password"]) {
            $_SESSION["user"] = $_POST["brugernavn"];
            $_SESSION["ip"] = $_SERVER["REMOTE_ADDR"];
        }else{
            $_SESSION["besked"] = $default_text;
        }
    }else{
    $_SESSION["besked"] = $default_text;
    }
}

?>

Side1.php

<?php
if (check_login()){
    print "<center><h2>Dette er side 1</h2></center>";
} else {
    include("./error.html");
    /* Vi kan også sende folk videre vha. header('location: error.html') */
}
?>

Hvorfor kan du ikke lide session

Damn har fundet fejlen.
Ved tidligere forsøg havde jeg bare skrevet include('settings.php');


<?php
include("settings.php");
if (check_login()){
    print "<center><h2>Dette er side 1</h2></center>";
} else {
    include("./error.html");
    /* Vi kan også sende folk videre vha. header('location: error.html') */
}
?>

Uanset hvad har i begge været en stor hjælp, jeg har lært en del af jeres svar.

Jeg springer over points.

-- Sessions er fine, men de er allerede låst til det SID (som enten er i en cookie, eller query-string), derfor er et IP check samtidigmed fjollet, da det ikke giver nogen som helst mening, og kan skabe fejl for folk som benytter tor eller har dynamisk ip.

Ahh, det kan jeg godt se...

Lukker