php - Hjælp med $_SESSION til login system - Udvikleren.dk

Tags: php

Bruger #12501 @ 27.11.07 17:22

Hej

Jeg skriver snart om hjælp i dette forum hver da, og her er et spørgsmål til:

Jeg har lavet et cms (content managment system) til balkanfestudvalget.dk, hvor man kan oprette, ændre og tilføje links, arrangementer og nyheder.

Dette er dog blevet spammet en del, da der kun var kodeord og brugernavns-beskyttelse på den første side. Jeg prøver nu at implementerer et loginsystem der virker over det hele ved hjælp af $_SESSION.

først har jeg en login side, der sender brugernavn og adgangskode videre via $_POST til oversigt.php. Her ser koden sådan ud:

Kode

<?php

session_start();

$brugernavn = "admin";
$kodeord = "bibliotek";

if (($_POST['brugernavn'] == $brugernavn and $_POST['kodeord'] == $kodeord) or ($_SESSION['login'] == "ACCEPTED")) {

$_SESSION['login'] = "ACCEPTED";

?>

...HER SKAL INDHOLDET VÆRE...

<?php

} else {

$_SESSION['login'] = "DENIED";

include("adgangnaegtet.php");

}

?>

Det virker fint med login'et der er videresendt via $_POST. $_SESSION['login'] bliver sat til "accepted", og hvis man klikker på opret ny nyhed, burde man blive sendt videre til addnyhed.php, som ser sådan ud:

Kode

<?php

session_start();

if ($_SESSION['login'] == "ACCEPTED") {

?>

...HER SKAL INDHOLDET VÆRE...

<?php

} else {

$_SESSION['login'] = "DENIED";

include("adgangnaegtet.php");

}

?>

Dette virker IKKE, og jeg får bare siden adgangnaegtet.php hver gang.

Nogle ideer?

Hilsen
/Aske

Bruger #10216 @ 27.11.07 18:34

4.283

Ved ikke hvad der går galt, men her får du lige en lidt mere struktureret udgave. Denne løsning består af tre filer.

Kode

<?php
	// login.php


	session_start();
	
	$brugernavn = "admin";
	$kodeord = "bibliotek";

	
	// Very secret stuff
	$mySecretString = 'Ich bin ein wienersnitzel';
	
	if (($_POST['brugernavn'] == $brugernavn and $_POST['kodeord'] == $kodeord) or ($_SESSION[md5($mySecretString)] == 'access')) {
		$_SESSION[md5($mySecretString)] = 'access';
	} else {
		$_SESSION[md5($mySecretString)] = NULL;
		unset($_SESSION[md5($mySecretString)]);
		
		// Gå til:
		header('location: login.php?error=1');
	}
	
	if(isset($_GET['error'])) {
		switch($_GET['error']) {
			case '1':
				print 'Fejl i login';
				break;
			case '2':
				print 'Du har ikke adgang. Log ind først';
				break;
			default:
				print 'Noget er gået galt';
		}
	}
	
	if($_SESSION[md5($mySecretString)] != 'access') {
		print '
			<form action="login.php" method="post">
				<input type="input" name="brugernavn" value="admin" />
				<input type="input" name="kodeord" value="bibliotek" />
				<input type="submit" name="login" value="Login" />
			</form>
			..HER SKAL INDHOLDET VÆRE...';
	}
?>







<?php
	// secure.php

	session_start();
	
	// Very secret stuff
	$mySecretString = 'Ich bin ein wienersnitzel';
	
	if($_SESSION[md5($mySecretString)] != 'access')) {
		// Gå til:
		header('location: login.php?error=2');
	}
?>







<?php
	// Alle andre filer
	
	// Security check
	require_once('secure.php');
	
	print ' ..HER SKAL INDHOLDET VÆRE...';
?>

Indlæg senest redigeret d. 27.11.2007 18:35 af Bruger #10216

Bruger #12501 @ 27.11.07 20:14

Ved ikke hvad der går galt, men her får du lige en lidt mere struktureret udgave. Denne løsning består af tre filer.

Kode

<?php
	// login.php


	session_start();
	
	$brugernavn = "admin";
	$kodeord = "bibliotek";

	
	// Very secret stuff
	$mySecretString = 'Ich bin ein wienersnitzel';
	
	if (($_POST['brugernavn'] == $brugernavn and $_POST['kodeord'] == $kodeord) or ($_SESSION[md5($mySecretString)] == 'access')) {
		$_SESSION[md5($mySecretString)] = 'access';
	} else {
		$_SESSION[md5($mySecretString)] = NULL;
		unset($_SESSION[md5($mySecretString)]);
		
		// Gå til:
		header('location: login.php?error=1');
	}
	
	if(isset($_GET['error'])) {
		switch($_GET['error']) {
			case '1':
				print 'Fejl i login';
				break;
			case '2':
				print 'Du har ikke adgang. Log ind først';
				break;
			default:
				print 'Noget er gået galt';
		}
	}
	
	if($_SESSION[md5($mySecretString)] != 'access') {
		print '
			<form action="login.php" method="post">
				<input type="input" name="brugernavn" value="admin" />
				<input type="input" name="kodeord" value="bibliotek" />
				<input type="submit" name="login" value="Login" />
			</form>
			..HER SKAL INDHOLDET VÆRE...';
	}
?>







<?php
	// secure.php

	session_start();
	
	// Very secret stuff
	$mySecretString = 'Ich bin ein wienersnitzel';
	
	if($_SESSION[md5($mySecretString)] != 'access')) {
		// Gå til:
		header('location: login.php?error=2');
	}
?>







<?php
	// Alle andre filer
	
	// Security check
	require_once('secure.php');
	
	print ' ..HER SKAL INDHOLDET VÆRE...';
?>

Hmm.. det virker sgu heller ikke. har lagt det op på serveren, og lavet et index.php med samme form som der ligger i login.php, men det virker ikke...

http://balkanfestudvalget.dk/test02/

Min side er hostet hos one.com. tror du det kan være noget de skal slå til før det virker???

Indlæg senest redigeret d. 27.11.2007 20:15 af Bruger #12501

Bruger #5620 @ 27.11.07 21:50

1.500

man skal passe på når man leger med header, man kan lave en evighedsløkke

, hvad Gnu såvidt jeg kan se har gjort

Kode

if (($_POST['brugernavn'] == $brugernavn and $_POST['kodeord'] == $kodeord) or ($_SESSION[md5($mySecretString)] == 'access')) {
		$_SESSION[md5($mySecretString)] = 'access';
	} else {
		$_SESSION[md5($mySecretString)] = NULL;
		unset($_SESSION[md5($mySecretString)]);
		
		// Gå til:
		header('location: login.php?error=1');
	}

hvis man ikke skriver det rigtige i login, redirecter den til sig selv og laver samme teste som fejler igen så redirecter igen og igen og igen...og igen.

Kode

    if(isset($_POST["login"])){
      if($_POST['brugernavn'] == $brugernavn and $_POST['kodeord'] == $kodeord){
		$_SESSION[md5($mySecretString)] = 'access';
	  }else{
		$_SESSION[md5($mySecretString)] = NULL;
		unset($_SESSION[md5($mySecretString)]);	
		// Gå til:
		header('location: login.php?error=1');
      }
	}

Bruger #10216 @ 27.11.07 21:56

4.283

aah ja... det gik vist lidt stærkt...

Godt spottet...

Bruger #11328 @ 27.11.07 23:29

1.323

Problemet kan vel også være i selve "check-sitet"...

Bruger #11328 @ 29.11.07 13:58

1.323

Ja altså det som nørden siger kan det være farligt at lege med headers... Jeg har selv lavet mange fejl med sådanne. Men kan ikk' rigtig se fejlen...

Hjælp med $_SESSION til login system

Karma barometer (30 dage)

Modtaget

Givet

Favorit hos
Forum tråde
Artikler