Hallo,

als beginner ben ik wat aan het stoeien met de beveiliging van website.

Ik heb een heel eenvoudig scriptje bijgevoegd (authenticatie1.inc.php) die via een require_once() wordt aangeroepen in een testscriptje (testaut.php).

In mijn studieboek ben ik ondertussen veel verder, dus ik weet dat het authenticatie script veel beter en uitgebreider kan, maar dat maakt voor mijn onderstaande vraag niets uit, want daar loop ik ook bij uitgebreidere scripts tegen aan.

Mijn vraag:
- stel je voert in het invulfomulier (authenticatie1.inc.php) een juiste naam + wachtwoord in, dan moet het scriptje verder lopen en de echo uitspugen ‘Je bent ingelogd’, maar dat gebeurt niet. Ik begrijp niet waarom dat niet gebeurd. ;
- ik zou zeggen dat het misgaat in het autheticatie1.inc.php in de regels 19-21. Ik zou zeggen dat er daar nog een terugverwijzing moet plaatsvinden van het autheticatiescript naar het testauthscript? Of gaat het ergens anders mis?

Kan het soms ook zijn dat het aan de instellingen van php.ini ligt wat betreft de opslag van cookies en session-variabelen? Voor de zekerheid post ik ook een stukje van de php.ini.

Wie zou me willen helpen?
Alvast bedankt, Pim


authenticatie1.inc.php

<?php
session_start();
function controle($naam, $wachtwoord) {
	if ($naam !="Pim" || $wachtwoord !=md5("pass100"))	{
		return false;
	} else {
		return true;
	}
}

//check naam en wachtwoord
if (isset($_POST["verzonden"]) && controle($_POST["naam"], md5($_POST["wachtwoord"]))==true) {
//naam en wachtwoord zijn beiden correct
	$_SESSION['naam'] = $naam;
	$_SESSION['wachtwoord'] = $md5($wachtwoord);
	}

if (controle($_SESSION['naam'],$_SESSION['$wachtwoord'])==false) {
//naam en/of wachtwoord is fout
	echo "<form method=\"post\" action=\"$PHP_SELF\">\n";
	echo "Naam: ";
	echo "<input type=\"text\" name=\"naam\">";
	echo "<br>";
	echo "Wachtwoord: ";
	echo "<input type=\"password\" name=\"wachtwoord\">";
	echo "<br>";
	echo "<input type=\"submit\" value=\"verzenden\" name=\"verzonden\">";
	echo "</form>";
	exit;
}
?>


testauth1.php

<?php
require_once('authenticatie1.inc.php');
echo 'je bent ingelogd';
?>


stukje php.ini

[Session]
; Handler used to store/retrieve data.
session.save_handler = files

; Argument passed to save_handler.  In the case of files, this is the path
; where data files are stored. Note: Windows users have to change this
; variable in order to use PHP's session functions.
;
; As of PHP 4.0.1, you can define the path as:
;
;     session.save_path = "N;/path"
;
; where N is an integer.  Instead of storing all the session files in
; /path, what this will do is use subdirectories N-levels deep, and
; store the session data in those directories.  This is useful if you
; or your OS have problems with lots of files in one directory, and is
; a more efficient layout for servers that handle lots of sessions.
;
; NOTE 1: PHP will not create this directory structure automatically.
;         You can use the script in the ext/session dir for that purpose.
; NOTE 2: See the section on garbage collection below if you choose to
;         use subdirectories for session storage
;
; The file storage module creates files using mode 600 by default.
; You can change that by using
;
;     session.save_path = "N;MODE;/path"
;
; where MODE is the octal representation of the mode. Note that this
; does not overwrite the process's umask.
;session.save_path = "/tmp"

; Whether to use cookies.
session.use_cookies = 1

;session.cookie_secure =

; This option enables administrators to make their users invulnerable to
; attacks which involve passing session ids in URLs; defaults to 0.
; session.use_only_cookies = 1

; Name of the session (used as cookie name).
session.name = PHPSESSID

; Initialize session on request startup.
session.auto_start = 0

; Lifetime in seconds of cookie or, if 0, until browser is restarted.
session.cookie_lifetime = 0

; The path for which the cookie is valid.
session.cookie_path = /

; The domain for which the cookie is valid.
session.cookie_domain =

; Whether or not to add the httpOnly flag to the cookie, which makes it inaccessible to browser scripting languages such as JavaScript.
session.cookie_httponly = 

; Handler used to serialize data.  php is the standard serializer of PHP.
session.serialize_handler = php

; Define the probability that the 'garbage collection' process is started
; on every session initialization.
; The probability is calculated by using gc_probability/gc_divisor,
; e.g. 1/100 means there is a 1% chance that the GC process starts
; on each request.

session.gc_probability = 1
session.gc_divisor     = 1000

; After this number of seconds, stored data will be seen as 'garbage' and
; cleaned up by the garbage collection process.
session.gc_maxlifetime = 1440

; NOTE: If you are using the subdirectory option for storing session files
;       (see session.save_path above), then garbage collection does *not*
;       happen automatically.  You will need to do your own garbage
;       collection through a shell script, cron entry, or some other method.
;       For example, the following script would is the equivalent of
;       setting session.gc_maxlifetime to 1440 (1440 seconds = 24 minutes):
;          cd /path/to/sessions; find -cmin +24 | xargs rm

; PHP 4.2 and less have an undocumented feature/bug that allows you to
; to initialize a session variable in the global scope, albeit register_globals
; is disabled.  PHP 4.3 and later will warn you, if this feature is used.
; You can disable the feature and the warning separately. At this time,
; the warning is only displayed, if bug_compat_42 is enabled.

session.bug_compat_42 = 0
session.bug_compat_warn = 1

; Check HTTP Referer to invalidate externally stored URLs containing ids.
; HTTP_REFERER has to contain this substring for the session to be
; considered as valid.
session.referer_check =

; How many bytes to read from the file.
session.entropy_length = 0

; Specified here to create the session id.
session.entropy_file =

;session.entropy_length = 16

;session.entropy_file = /dev/urandom

; Set to {nocache,private,public,} to determine HTTP caching aspects
; or leave this empty to avoid sending anti-caching headers.
session.cache_limiter = nocache

; Document expires after n minutes.
session.cache_expire = 180

; trans sid support is disabled by default.
; Use of trans sid may risk your users security.
; Use this option with caution.
; - User may send URL contains active session ID
;   to other person via. email/irc/etc.
; - URL that contains active session ID may be stored
;   in publically accessible computer.
; - User may access your site with the same session ID
;   always using URL stored in browser's history or bookmarks.
session.use_trans_sid = 0

; Select a hash function
; 0: MD5   (128 bits)
; 1: SHA-1 (160 bits)
session.hash_function = 0

; Define how many bits are stored in each character when converting
; the binary hash data to something readable.
;
; 4 bits: 0-9, a-f
; 5 bits: 0-9, a-v
; 6 bits: 0-9, a-z, A-Z, "-", ","
session.hash_bits_per_character = 5

; The URL rewriter will look for URLs in a defined set of HTML tags.
; form/fieldset are special; if you include them here, the rewriter will
; add a hidden <input> field with the info which is otherwise appended
; to URLs.  If you want XHTML conformity, remove the form entry.
; Note that all valid entries require a "=", even if no value follows.
url_rewriter.tags = "a=href,area=href,frame=src,input=src,form=fakeentry"




Op regel 15 heb je een dollarteken voor je md5 staan. Ik denk dat hij zal zoeken naar een variabele $md5, en de waarde daarvan als de naam van de functie die hij moet aanroepen zal gebruiken. Dat zou een grote error moeten opleveren.

Ook zou ik van regel 18 een elseif maken. Nu kijk je of er wordt ingelogd, maar onafhankelijk daarvan kijk je ook of er een sessie is. Beetje dubbelop in het geval dat er iemand handmatig inlogt.

En $PHP_SELF is verouderd, en bestaat niet altijd meer. Probeer eens $_SERVER['PHP_SELF']. Gewoon het action attribuut leeg laten zou ook prima moeten werken overigens. De browser pakt dan vanzelf wel de huidige URL.
Heb je soms een oud boek?

Waar komt $naam vandaan? Regel 14 eerste script.
Idem $wachtwoord op regel 15.
Regel 18 => '$wachtwoord'?
Regel 20 => $PHP_SELF? => $_SERVER['PHP_SELF']
Doe de echo's met enkele quotes dan hoef je de html-quotes niet te escapen. $Vars altijd buiten de quotes houde.
echo "<form method=\"post\" action=\"$PHP_SELF\">\n";
wordt dan
echo '<form method="post" action="' . $SERVER['PHP_SELF'] . '">' . "\n"; // \n moet altijd tussen dubbele quotes.
@Jelmer

je advies opgevolgd, en het werkt nu.

Het ligt vooral aan de regel 28, de 'if' verandert in 'elseif'. ik kon dus gewoon niet handmatig inloggen.

Reuze bedankt voor je hulp, daar krijg ik weer een opkikker van!!!!

groetjes, Pim
@ SanThe,

dank voor je reactie,

het boekje waaruit ik studeer heet 'Leer jezelf professioneel werken met ..> PHP5MySQL, zoals ik heb vermeld op de boekenpagina van deze site.

Het beekje dateert van okt. 2005 (volgens de ondersteunende site: www.leer-php.nl), maar in het boekje zelf staat 2006.

Frapant dat jij en Jelmer er kleine foutjes uithalen, want de auteur werkt (volgens zijn site) bij Eduvision als PHP docent en dan verwacht je toch foutloze en actuele code.

Zeker jullie opmerkingen t.a.v. het gebruik van $_SERVER['PHP_SELF] i.p.v. $PHP_SELF en het escapen van de html-quotes.

Je vraagt waar in regel 14/15 de $naam en $wachtwoord vandaan komen? Ik neem aan dat je bedoelt dat die eigenlijk eerst zou moeten worden gedefiniëerd, met zo iets als:
$naam='Pim';
$wachtwoord='pass120';
, maar ik ging ervan uit dat de auteur het voldoende vond dat ze in regel 4 (function) zijn ondergebracht (of dat ze in een apart aanmeldscript zouden staan (maar dat staat niet in het boekje erbij vermeld)).

Wat vind je ervan?
groet Pim
Zet maar eens error reporting aan, dan zul je notices krijgen van jouw undefined variabelen.
Pim schreef op 05.08.2007 14:51
Frapant dat jij en Jelmer er kleine foutjes uithalen, want de auteur werkt (volgens zijn site) bij Eduvision als PHP docent en dan verwacht je toch foutloze en actuele code.

Zeker jullie opmerkingen t.a.v. het gebruik van $_SERVER['PHP_SELF] i.p.v. $PHP_SELF en het escapen van de html-quotes.

Jelmer en SanThe zijn dan ook steengoed he ;-)

Boeken van twee jaar oud kun je vaak beter niet meer helemaal vertrouwen. PHP is een taal in ontwikkeling en om bij te blijven zit je op dit forum wel op de goede plek.
GaMer13 schreef op 05.08.2007 14:53
Zet maar eens error reporting aan, dan zul je notices krijgen van jouw undefined variabelen.

<?php
ini_set('display_errors', 1);
error_reporting(E_ALL | E_STRICT);

// rest van je script
?>
Zodra je de website online zet, verander je de 1 in een 0 en bezoekers kunnen onmogelijk foutmeldingen op het scherm krijgen. Dat wil je niet i.v.m. beveiliging.

Uiteraard heb je een eigen error-handler die fouten wegschrijft in een logboek. Dan kun je later nog eens oplossen.
Even een copy-past waarmee je iets beter uit de voeten kunt;

<?php
$valid_passwords = array ("mario" => "carbonell");
$valid_users = array_keys($valid_passwords);

$user = $_SERVER['PHP_AUTH_USER'];
$pass = $_SERVER['PHP_AUTH_PW'];

$validated = (in_array($user, $valid_users)) && ($pass == $valid_passwords[$user]);

if (!$validated) {
header('WWW-Authenticate: Basic realm="My Realm"');
header('HTTP/1.0 401 Unauthorized');
die ("Not authorized");
}

// If arrives here, is a valid user.
echo "<p>Welcome $user.</p>";
echo "<p>Congratulation, you are into the system.</p>";

?>
@pgfrank

je kan natuurlijk ook error_reporting(1); gebruiken, dan geldt het alleen voor die ene pagina
timmy schreef op 05.08.2007 16:57
@pgfrank

je kan natuurlijk ook error_reporting(1); gebruiken, dan geldt het alleen voor die ene pagina
En waar zou dat handig voor zijn? Kun je later tientallen scripts gaan aanpassen wanneer je de site online zet.

Zet de instellingen in een config_file die je overal include en je hebt het voor de complete site in 1 keer goed ingesteld.

Reageren