LINUXSOFT.cz Přeskoč levou lištu
Uživatel: Heslo:  
   CZUKPL

> PHP (84) - Ještě k registraci

Chybovati je lidské, aneb proč se někteří uživatelé nemohli na náš cvičný portál registrovat.

29.12.2004 15:00 | Petr Zajíc | Články autora | přečteno 30415×

Dnes budeme řešit záležitost mimo plán - i takové věci se při vývoji software stávají. Budeme řešit reálný problém, který vznikl s registrací na tomto cvičném portálu. Je to naléhavé, protože většina věcí bez funkčního přihlašování nejde vyzkoušet.

Problém s přihlašováním

Pravděpodobně si vzpomínáte, že registrační skript generoval e-mail, který obsahoval instrukce k dokončení registrace. Vtip spočívá v tom, že uživatel si musí e-mail stáhnout a zařídit se podle instrukcí v něm obsažených. To v našem případě reprezentuje kliknutí na odkaz. A ten vede na stránku pro dokončení registrace. Pro osvěžení kód, který mail generuje:

  $path="http://".$_SERVER["SERVER_NAME"].":".$_SERVER["SERVER_PORT"].$_SERVER["SCRIPT_NAME"].
  
"?clanek=registracedokoncena&id=".$lastid."&email=".$_POST["email"];
  
$telo = "Dekujeme Vam za registraci na nasem portale [nazev hudebniho portalu]. Pro dokonceni ".
  
"registrace klepnete na nasledujici odkaz\n<a href=\"$path\">$path</a>\n".
  
"Pokud jste se na nasem portale nechteli zaregistrovat, povazujte tento e-mail za bezpredmetny.\n".
  
"Vas [nazev hudebniho portalu] team.";
  @
mail ($_POST["email"], "Registrace na portalu", $telo);

V kódu nebylo nic závadného, a nic závadného nebylo ani v kódu, který registraci dokončuje:

  mysql_query("update uzivatele set registracedokoncena=1 where id=".$_GET["id"]." and email='".$_GET["email"]."';",$GLOBALS["link"]);
  echo
"Vaše registrace byla dokončena."

Přes to přeze všechno kód některým uživatelům nefungoval. Jak se zdá, je to proto, že v proměnné $path je použito znaků, &amp; jako entity ampersandu. Je tomu tak proto, že to vyžaduje norma a je to jistě správně. Někteří e-mailový klienti však mají se zpracováním takto napsaných odkazů problém a hodnoty parametrů clanek, id a email jsou předávány špatně nebo vůbec.

Řešení jsou v podstatě tři. Je možné:

  1. Místo &amp; psát pouze ampersand. To nechceme, není to podle normy.
  2. Přepsat portál tak, aby odkaz na registraci nevyžadoval. To je v našem případě poměrně problematické, protože by to znamenalo změnit celou logiku zpracování registrace.
  3. Přestat používat ve zprávě HTML tagy a posílat ji jako prostý text

My se pustíme tím třetím směrem a přepíšeme text zprávy tak, aby neobsahoval žádné HTML tagy. Neboli, ze zprávy ve formátu HTML uděláme zprávu ve formátu "prostý text". Sice to znamená, že uživatel nebude moci pohodlně dokončit registraci kliknutím na odkaz ve svém e-mailovém klientovi, ale:

  • Zato to bude fungovat se všemi e-mailovými klienty včetně těch, kteří odkazy nepodporují
  • Většina pošťáků si stejně bude hrát na chytré a odkaz z toho vytvoří. Doufejme, že správný.

Chyba plodí chybu

Aby toho nebylo málo, ukázalo se zároveň, že informační hláška po dokončení registrace "Vaše registrace byla dokončena" je v případě problémů s posíláním parametrů v odkazech v podstatě nesmyslná. Informuje nás totiž o něčem, co není pravda. Abychom měli jistotu, že se dotaz skutečně podařilo v databázi provést, měli bychom udělat nějaké testy. Tak například bychom mohli testovat, zda se tím příkazem změnil právě jeden řádek v databázi. Pokud se tak nestalo (nejspíš proto, že dotaz neaktualizoval žádný řádek), neměli bychom tvrdit, že "registrace byla dokončena". Úprava může spočívat v následujícím kódu:

  if (mysql_affected_rows()==1) echo "Vaše registrace byla dokončena.";
  else echo
"Příkaz nebyl proveden, zřejmě jsou nesprávné parametry nebo již registrace byla dokončena";

Funkce mysql_affected_rows se zhruba podobná funkci mysql_num_rows až na to, že se používá pro zjišťování počtu řádků ovlivněných akčními dotazy, kdežto mysql_num_rows je pro počet řádků vrácených výběrovými dotazy.

Pozn.: Akční dotazy jsou takové, které mění stávající data. Jako například INSERT, UPDATE, DELETE a některé další. Výběrové dotazy jsou ty, začínající slovem SELECT.

Je to bezpečné?

Tak, chyba byla zažehnána. Když už jsme u registrace, nabízí se ještě jedna otázka: Je posílání identifikačního čísla a e-mailu uživatele bezpečné? Diplomatická odpověď je ta, že ne až tak úplně. Při troše důvtipu by totiž mohl útočník vyzkoušet různé kombinace parametrů id a při troše štěstí by se mu mohlo povést dokončit registraci i v případě, že by daný e-mail vůbec neexistoval! Navíc by každý, kdo by se dostal k přenášené poště mohl ze zprávy vyčíst identifikátor a e-mailovou adresu právě přihlašovaného uživatele.

Pozn.: Nám by to na testovacím portále sice zase až tak nevadilo, ale v praxi by mohlo být ověření funkční adresy pro portál klíčové a možnost zaregistrovat se bez uvedení platného e-mailu by mohla mít fatální důsledky. Stejně fatální by mohlo být vyluštění citlivých informací z e-mailu.

Tak či tak, nastíníme si řešení, které má oproti našemu současnému dvě podstatné výhody:

  1. Namísto dvou parametrů id a email se skriptu posílá pouze jeden parametr. Ne snad, že by to vyřešilo problémy s ampersandy ve zprávě, ale bude to kompaktnější.
  2. Přenášený údaj bude zašifrovaný a bude mu rozumět zase jen registrační skript

Dosáhnout toho není zas až tak složité, jak by se mohlo na první pohled zdát. Pokud využijeme poznatky z 51.dílu našeho seriálu o předávání dat pomocí parametrů, máme prakticky vystaráno. Kód by mohl vypadat nějak takto:

  $sifra= new Crypt_Xtea;
  
$citlive=$lastid."|".$_POST["email"];
  
$citlive=urlencode(base64_encode($sifra->encrypt((string)$citlive, "[šifrovací klíč]")));

Telegraficky řečeno využívá se šifrování pomocí třídy Crypt_Xtea. Parametry se spojí do jednoho a zašifrují. Použití znaku "|" jako oddělovače je bezpečné, protože ani identifikátor ani e-mail jej nebudou obsahovat. Skript pro dokončení registrace pak může vypadat nějak takto:

  $sifra= new Crypt_Xtea;  
  
$citlive=$sifra->decrypt(base64_decode(urldecode($_GET["id"])),"[šifrovací klíč]");
  
$udaje=explode("|",$citlive);

... a následuje zápis do databáze, jak jej známe. Přičemž klíč musí být pochopitelně stejný jako ten, který jsme použili při šifrování. Proč jsou tam zahrnuty funkce base64_encode, base64_decode, urlencode a urldecode se dozvíte v 51. díle, kde je to rozebráno podrobněji. Zkráceně řečeno je to proto, aby se řetězec bezpečně přenesl jako text, přestože může být binární.

Poděkování za objev "zakopaného psa" patří Honzovi Houšťkovi, který přišel na to, že problémy při dokončování registrace způsobují mechanizmy, kterými někteří e-mailoví klienti zpracovávají odkazy v HTML zprávách.

Úloha komunity

V tomto bodě se hodí připomenout výhody sdílení kódu s jinými vývojáři, snad pod nějakou svobodnou licencí. Mají-li totiž ostatní přístup k tomu, co vyvíjíte, mohou se podobné problémy objevit a vyřešit mnohem pravděpodobněji, než když na projektu pracuje jeden vývojář nebo jen uzavřená skupina lidí. Proto byste při vývoji vlastního software měli minimálně uvažovat o tom, zda a za jakých podmínek by bylo možné zpřístupnit vaši práci ostatním vývojářům. GNU/Linux je na tomto principu založen a funguje. Nemá to nic společného s tím, zda za svou práci budete chtít peníze či nikoli; více se můžete dozvědět studiem příslušných licencí. Každopádně pozitivní vliv spolupráce více lidí při tvorbě kódu je nesporný.

Změny na portále

Především, všechny nedokončené žádosti o registraci byly z databáze odstraněny, takže se můžete znovu registrovat, pokud to předtím nefungovalo. Byly změněny soubory registrace.php a registracedokoncena.php. A byly přidány moduly pro šifrování. Na současný stav portálu se na našem webu můžete podívat, nebo si jej můžete stáhnout ve formátu zip.

Verze pro tisk

pridej.cz

 

DISKUZE

update 0 - 1 4.1.2007 08:39 onza
  L Re: update 0 - 1 4.1.2007 09:23 onza




Příspívat do diskuze mohou pouze registrovaní uživatelé.
> Vyhledávání software
> Vyhledávání článků

23.5.2018 20:55 /Ondřej Čečák
Od pátku 25.5. proběhne na Fakultě informačních technologií ČVUT v Praze openSUSE Conference. Můžete se těšit na spostu zajímavých přednášek, workshopů a také na Release Party nového openSUSE leap 15.0. V na stejném místě proběhne v sobotu 26.5. i seminář o bezpečnosti CryptoFest.
Přidat komentář

20.5.2018 17:45 /Redakce Linuxsoft.cz
Ve čtvrtek 31. května 2018 připravuje webový magazín BusinessIT ve spolupráci s Best Online Média s.r.o. pátý ročník odborné konference Firemní informační systémy 2018. Akce proběhne v kongresovém centru Vavruška (palác Charitas), Karlovo náměstí 5, Praha 2 (u metra Karlovo náměstí) od 9:00 hod. dopoledne do cca 15 hod. odpoledne. Konference je zaměřena na efektivní využití firemních informačních systémů a na to, jak plně využít jejich potenciál. Podrobnější informace na webových stránkách konfrence.
Přidat komentář

14.5.2018 7:28 /František Kučera
Květnový pražský sraz spolku OpenAlt se koná již tento čtvrtek – 17. 5. 2018 od 18:00 v Kavárně Ideál (Sázavská 30, Praha), kde máme rezervovaný salonek. Tentokrát na téma: Audio – zvuk v GNU/Linuxu.
Přidat komentář

7.5.2018 16:20 /František Kučera
Na stránkách spolku OpenAlt vyšla fotoreportáž Pražské srazy 2017 dokumentující srazy za uplynulý rok. Květnový pražský sraz na téma audio se bude konat 17. 5. 2018 (místo a čas ještě upřesníme).
Přidat komentář

17.4.2018 0:46 /František Kučera
Dubnový pražský sraz spolku OpenAlt se koná již tento čtvrtek – 19. 4. 2018 od 18:00 v Kavárně Ideál (Sázavská 30, Praha), kde máme rezervovaný salonek. Tématem tohoto srazu bude OpenStreetMap (OSM) aneb svobodné mapy.
Přidat komentář

16.3.2018 22:01 /František Kučera
Kulatý OpenAlt sraz v Praze oslavíme klasicky: u limonády a piva! Přijďte si posedět, dát si dobré jídlo a vybrat z mnoha piv do restaurace Kulový blesk, který najdete v centru Prahy nedaleko metra I. P. Pavlova na adrese Sokolská 13, Praha 2. Sraz se koná ve čtvrtek 22. března a začínáme v 18:00. Heslo: OpenAlt. Vezměte s sebou svoje hračky! Uvítáme, když si s sebou na sraz vezmete svoje oblíbené hračky. Jestli máte nějaký drobný projekt postavený na Arduinu, nějakou zajímavou elektronickou součástku, či třeba i pěkný úlovek z crowdfundingové akce, neváhejte. Oslníte ostatní a o zábavu bude postaráno.
Přidat komentář

13.2.2018 0:41 /František Kučera
Únorový pražský sraz OpenAltu se koná 15. 2. 2018 a tentokrát se vydáme na návštěvu do jednoho pražského datacentra. Sejdeme se v 17:50 v severovýchodní části nástupiště tramvajové zastávky Koh-I-Noor. Po exkurzi se přesuneme do restaurace U Pštrosa (Moskevská 49), kde probereme tradiční témata (svobodný software a hardware, DIY, CNC, SDR, 3D tisk…) a tentokrát bude k vidění i IoT brána od The Things Network.
Přidat komentář

11.2.2018 23:11 /Petr Ježek
Hledáte lehký a rychlý prolížeč PDF souborů? Pokud vás již omrzelo čekat na načítání stránek či jiné nešvary, zkuste xreader.
Přidat komentář

   Více ...   Přidat zprávičku

> Poslední diskuze

20.2.2018 18:48 / Ivan Majer
portal

20.2.2018 15:57 / Jan Havel
Jak využíváte služby cloudu v podnikání?

16.1.2018 1:08 / Ivan Pittner
verejna ip od o2 ubuntu

15.1.2018 17:26 / Mira Harvalik
Re: Jak udělat HTML/Javascript swiping gallery do mobilu?

30.12.2017 20:16 / Michal Knoll
odmocnina

Více ...

ISSN 1801-3805 | Provozovatel: Pavel Kysilka, IČ: 72868490 (2003-2018) | mail at linuxsoft dot cz | Design: www.megadesign.cz | Textová verze