Skrevet tir. d. 01. februar 2011 kl. 11:50:25| #1

nemlig (5.151 point)

<?php
// Log ind
if (isset($_SESSION["username"]) && isset($_SESSION["userid"])) {
  if ($_SESSION["username"] == '1') {
    echo "<a href='/php/opret.php'>Opret Bruger</a>";
  }
  echo "<a href='/logud.php'>Log ud</a>";
} else {
  echo "<a href='/logind.php'>Log ind</a>";
}
?>

Anmeld misbrug

Skriv et svar | Skriv en kommentar

Skrevet tir. d. 01. februar 2011 kl. 11:52:18| #2

nemlig (5.151 point)

Så går jeg ud fra, at session-variablen "username" = admin.
Hvor kommer "admin" ellers ind i billedet?
Er det en brugertype, og hvis ja, hvordan er den defineret.

Anmeld misbrug

Skriv et svar | Skriv en kommentar

Skrevet tir. d. 01. februar 2011 kl. 11:54:40| #3

splazz (57.138 point)
majbom.com

og så sørg lige for at lave et tjek på opret.php, så hvis uautoriserede personer skulle finde derind ikke får adgang :)

Anmeld misbrug

Skriv et svar | Skriv en kommentar

Skrevet tir. d. 01. februar 2011 kl. 12:36:09| #4

PFPedersen (nedlagt brugerprofil) (2.855 point)

Admin er et felt i bruger db der angives med 1 hvis aktiv, og som jeg skal sætte manuelt. Det er ikke så avanceret. Jeg retter det nok til så det kan sættes automatisk ved oprettelse.

Man kan ikke opnå adgang til opret bruger uden at være logget ind, håber det er nok til at holde andre væk. Og med de her svar fra jer, vil det også kræve admin for at få adgang til opret.

Har læst et sted at man bør mistænke alle sine brugere, og dermed sikre alt optimalt :)

Anmeld misbrug

Skriv et svar | Skriv en kommentar

Skrevet tir. d. 01. februar 2011 kl. 12:42:39| #5

PFPedersen (nedlagt brugerprofil) (2.855 point)

Svar til Nemlig

Anmeld misbrug

Skriv et svar | Skriv en kommentar

Skrevet tir. d. 01. februar 2011 kl. 12:47:14| #6

splazz (57.138 point)
majbom.com

præcis, derfor bør du også tjekke brugere der er logget ind om de rent faktisk har adgang til siden :)

Anmeld misbrug

Skriv et svar | Skriv en kommentar

Skrevet tir. d. 01. februar 2011 kl. 12:48:31| #7

nemlig (5.151 point)

Jeg er i tvivl om, du har styr på koden.
Admin-feltet har du måske også i en session, og så kan koden se sådan her ud:

<?php
// Log ind
if (isset($_SESSION["username"]) && isset($_SESSION["userid"])) {
  if ($_SESSION["admin"] == '1') {
    echo "<a href='/php/opret.php'>Opret Bruger</a>";
  }
  echo "<a href='/logud.php'>Log ud</a>";
} else {
  echo "<a href='/logind.php'>Log ind</a>";
}
?>

Anmeld misbrug

Skriv et svar | Skriv en kommentar

Skrevet tir. d. 01. februar 2011 kl. 13:32:02| #8

PFPedersen (nedlagt brugerprofil) (2.855 point)

Jeg har ikke ret meget styr på php generelt, jeg kan til nøds finde rundt i en skreven kode, men jeg kan slet ikke skrive det selv :(

Jeg fandt ud af det med admin session, for først havde jeg "kopieret" en sætning i øverste linje

<?php
// Log ind
if (isset($_SESSION["username"]) && isset($_SESSION["userid"]) && isset($_SESSION["admin"])) {

Men det resulterede i at den anden 'ikke-admin' bruger ikke kunne logge ind.

Tak for hjælpen :)

Anmeld misbrug

Skriv et svar | Skriv en kommentar

Skrevet tir. d. 01. februar 2011 kl. 13:41:51| #9

nemlig (5.151 point)

Jeg tror mit forslag til kode vil virke for dig.

Anmeld misbrug

Skriv et svar | Skriv en kommentar

Skrevet tir. d. 01. februar 2011 kl. 13:49:05| #10

PFPedersen (nedlagt brugerprofil) (2.855 point)

Det gør det også, det var min første idé der ikke virkede :)

Anmeld misbrug

Skriv et svar | Skriv en kommentar