Sleep() in login

Maaark

Ja inderdaad, alleen ben ik wat te lui om alles netjes erneer te zetten... Daarnaast is een LIMIT ook handig om erin te hebben, scheelt weer een beetje.. (alle beetjes helpen heaaa)

YVMdesign

Wat kan je beter gebruiken?

PHP

if ($_SERVER['REQUEST_METHOD'] == 'POST')
{

of dit

PHP

if (isset($_POST['submit']))
{

Maaark

Bovenste

YVMdesign

Dankje :cheer:

Maaark

Post je na afloop ook je gehele script nog eens?
Hebben andere leden er ook nog wat aan

YVMdesign

Ik heb dit tot nu toe.

PHP

require("config.php");


if ($_SERVER['REQUEST_METHOD'] == 'POST')
{
	// use sleep for anti brute-force
	sleep(1);
	
	// with mysql_real_escape_string
	$nickname = mysql_real_escape_string($_POST['nickname']);
	$password = mysql_real_escape_string($_POST['password']);
	
	$query = mysql_query("SELECT nickname FROM Members WHERE nickname='" .stripslashes($nickname). "'");
	
	if (empty($nickname) OR empty($password))
	{
		header("Refresh: 5; url=index.php");
		die("You forgot something.");
	}
	elseif (mysql_num_rows($query) == 0)
	{
		header("Refresh: 5; url=index.php");
		die("This nickname doesn't exist.");
	}
	else
	{
		$query = mysql_query("SELECT nickname, password, onlinetime FROM Members WHERE nickname='" .stripslashes($nickname). "'");
		$result = mysql_fetch_assoc($query);
		
		if (md5($password) == $result['password'])
		{
			$_SESSION['nickname'] = $nickname;
			mysql_query("UPDATE Members SET onlinetime='"  .time(). "' WHERE nickname='" .addslashes($nickname). "'");
			header("Refresh: 5; url=index.php");
			die("You are logged in, please wait 5 secondes.");
		}
		else
		{
			header("Refresh: 5; url=index.php");
			die("Wrong password.");
		}
	}
}

Meer tonen

Misschien nog wat opmerkingen??

Maaark

Je zou bijvoorbeeld LIMIT er nog in kunnen verwerken en zoals ik al zei trim.
En van de bovenste query niet nickname maar id ofzo selecten, die is wat kleiner.

YVMdesign

Trim erin verwerkt

Wat bedoel je met LIMIT? (voorbeeld misschien?)

PHP

elseif (trim($nickname) OR trim($password))
	{
		header("Refresh: 5; url=login.php");
		die("Invalid input.");
	}

Zo toch?

Maaark

PHP

$query = mysql_query("SELECT nickname, password, onlinetime FROM Members WHERE nickname='" .stripslashes($nickname). "' LIMIT 1");

Darsstar

Spoiler/waarschuwing:
Ik ga boos op je worden, mogelijk wil je dit bericht niet lezen.

Ok...
*wordt boos*
stripslashes()?
*wordt bozer*
Wat heeft stripslashes in Ori's naam met backticks te maken?
Je bent nu mysql_real_escape_string() ongedaan aan het maken!
*is behoorlijk boos*
Verder heb ik zelf niets tegen backticks.
Wel is het zo dat je reserved keywords zoveel mogelijk (altijd eigenlijk) moet vermeiden.
Dat ga je dus onmiddellijk ongedaan maken, begrepen!?!

Waarom al een query uitvoeren voordat je zelfs maar de input hebt gechecked?
Kijk eerst of een of meer velden niet ingevuld zijn, en ga dan pas een query sturen naar de database.
Ook kan de wachtwoord controle makkelijk in de query geplaatst worden.

edit:
verder wil je trim() binnen empty() zetten...
nu wordt je doorgestuurd als je het formulier juist wel invuld...

Axl_H

[offtopic] Darsstar: denk aan je bloeddruk:P[/offtopic]

Je kan bij je md5 ook dit er van maken:

PHP

<?php 


md5(sha1($_POST['veld dat je post']));


?>

Darsstar

Axl_H
Zo creëer je alleen maar meer collisions.
Het wordt wel veiliger door het volgende:

PHP

<?php 


$hash = md5($_POST['veld dat je post']).sha1($_POST['veld dat je post']);


?>

ICTscripters

Dé plek voor IT

Dé plek voor IT

nieuwe game

Criminolz

[Beta] Ottomaan: gratis browserspel met een echte, levende stad. Feedback gezocht!

Port Father: beta van een browsergame in Antwerpen 1931, feedback welkom

Maffia Hotel: een browsergame waarin je van lobbygast naar spil van het hotel groeit

Crime Club volledig herschreven vanaf nul

Malavita (nieuwe maffia game, van 0)

Herkent deze Notion-structuurscan echte problemen?

[Aanbod] Stadskaart voor je browsergame: van menu naar klikbare stad, vanaf €200

[Verkoop] G4G (Gangsters 4 Gangsters): volledige broncode, nieuw gebouwd in PHP 8.3 | € 30 per domein

[TE KOOP] Malavita.be [Maffiaspel, gebaseerd op omicidio.nl, unieke code]

Sleep() in login

Registratie

PHP waarschuwt voor twee ernstige bugs die makkelijk uit te buiten lijken

Functioneel ontwerp

Access Control List implementatie in PHP/MySQL - deel 1/2

Access Control List implementatie in PHP/MySQL - deel 2/2

Gebruikers die dit topic bekijken