Lista alla användare

12 svar · 323 visningar · startad av Renault 25 juli 2002

RenaultMedlem sedan nov. 2001101 inlägg

Frågan25 juli 2002 10:25#1

Sedan jag loggat in och sedan vill titta på användare så listas bara mitt användarnamn och användare som har högre idnummer!

Scriptet ser ut så här:

<% Visa = "SELECT * FROM users" %>

\<table border="0" cellpadding="0" cellspacing="0"\>

\<tr\>
\<td width="200"\>\<font size="2" face="verdana, arial"\>\<b\>Användare\</b\>\</font\>\</td\>
\<td width="200"\>\<font size="2" face="verdana, arial"\>\<b\>Lösenord\</b\>\</font\>\</td\>
\</tr\>

<% Do While Not RecSet.EOF %>

\<tr\>
\<td\>\<font size="2" face="verdana, arial"\>\<% =RecSet("Username") %\>\</font\>\</td\>
\<td\>\<font size="2" face="verdana, arial"\>\<% =RecSet("Password") %\>\</font\>\</td\>
\</tr\>

<% RecSet.MoveNext
Loop %>

<% RecSet.Close %>

Finns lösning som listar alla användare?

@ndersMedlem sedan juni 200032 967 inlägg

25 juli 2002 10:29#2

Det är inget fel på koden. Det enda jag kan tänka mig är att det endast finns användare med högre ID-nummer än du...

LAMPMedlem sedan mars 2002777 inlägg

25 juli 2002 10:32#3

Det enda felet jag hittar är att du aldrig gör en execute på din sql-fråga. Men eftersom du får ut resultat har du säkert bara glömt det i inlägget.

@ndersMedlem sedan juni 200032 967 inlägg

25 juli 2002 10:34#4

Att jag missade det också. Bit mig! Stånga mig! Gör något!

Tur att någon har ögonen med sig i alla fall... ;)

Ditt recordset innehåller alltså något annat. Vad har du fyllt det med tidigare? Ingen inloggningshistoria à la IDG hoppas jag?

Mvh,

RenaultMedlem sedan nov. 2001101 inlägg

25 juli 2002 11:10#5

Loggar jag in med idnummer 1 kommer alla med, annars inte!
Det gick just upp ett ljus för mig. Ska kolla en sak - återkommer!

Förresten, ska inte "inloggningshistoria à la IDG" vara att rekommendera? För mig har det funkat bra, tills nu!

@ndersMedlem sedan juni 200032 967 inlägg

25 juli 2002 11:15#6

Inloggningshistoria à la IDG är absolut inte att rekommendera.

* Användare A kan logga in med vilket lösenord som helst som finns i användartabellen.
* Hela tabellen loopas igenom för att hitta matchande fält, istället för att göra jämförelsen i SQL-frågan.
* IDGs inloggningsscript är kladdigt och onödigt mycket kod. Det räcker med 5 rader för att göra ett script som är åtminstone två miljoner gånger bättre och säkrare än IDGs.

Du kan ju roa dig lite med att söka på idg här i ASP-forumet om du vill... ;)

Mvh,

G8Medlem sedan jan. 20011 213 inlägg

25 juli 2002 11:36#7

Hmm, intressant, hur ser IDG's kod ut jamnfort med den sakrare som du snackar om @nders?

Vill bara kolla sa att jag inte maste gora om hela mitt egan loginscript... ;)

Exemple nagon?

@ndersMedlem sedan juni 200032 967 inlägg

25 juli 2002 11:43#8

http://www.webforum.nu/showthread.php?s=&threadid=8218&highlight=idg+login

G8Medlem sedan jan. 20011 213 inlägg

25 juli 2002 11:52#9

Ja, det ser jag ju att den inte ar sa stabil...

WHERE UserName = '" & Request.Form("Username") & "' AND Password = '" & Request.Form("Password") & "'"

Kan ju vara bra att ha med ;)

Dumma IDG, tur att man inte har lart sig genom de iaf :e

@ndersMedlem sedan juni 200032 967 inlägg

25 juli 2002 11:54#10

En sak:

Man ska inte glömma bort att escape:a apostrofer från användarnamns- och lösenordsfälten. Båda ur säkerhetsaspekt och för att tillåta det tecknet i användarnamn och lösenord. :-)

Mvh,

RenaultMedlem sedan nov. 2001101 inlägg

25 juli 2002 12:49#11

-> @nders

Man ska inte glömma bort att escape:a apostrofer från användarnamns- och lösenordsfälten. Båda ur säkerhetsaspekt och för att tillåta det tecknet i användarnamn och lösenord. :-)

Hur löser jag detta?

@ndersMedlem sedan juni 200032 967 inlägg

25 juli 2002 12:50#12

strUserName = Replace(Request.Form("UserName"), "'", "''")

använd alltid escape:ade strängar i dina SQL-frågor.

Mvh,

RenaultMedlem sedan nov. 2001101 inlägg

25 juli 2002 13:05#13

Tackar för lektionen