1. PHP / Говнокод #8225

    +165

    1. 1
    $idlink = rawurlencode(base64_encode($name . "||" . $email . "||" . md5( $password1 ) . "||" . $accountID_1 . "||" . $accountID_2 . "||" . $accountID_3 . "||" . $serverid_1 . "||" . $serverid_2 . "||" . $serverid_3 . "||" . md5( sha1( $name . $email . DBHOST . DBNAME . $config['key'] ) ) ) );

    Значит так, ребята. Готовьтесь воспринимать. Данные с регистрационной формы кодируются как только можно, потом с этим кодом генерируется ссылка и отправляется письмо с подтверждением регистрации. Когда новый пользователь переходит по ссылочке, данные берутся из базы, снова кодируются и сравниваются с кодом из ссылки.

    Пы.Сы. Код не мой

    Запостил: kindofbear, 18 Октября 2011

    Комментарии (43) RSS

    • ava sectus 18.10.2011 08:24 # 0

      и чЁ?
      Ответить

      • ava kindofbear 18.10.2011 08:26 # 0

        Анальное совокупление, на мой взгляд. Куча кодирований
        И вообще - use bcrypt
        Ответить

        • ava sectus 18.10.2011 08:36 # 0

          В общем - да... вопрос тут можно поставить так: Нахрена?
          Ответить

        • ava Teddy_Brown 18.10.2011 12:03 # +1

          я бы сделал так : 
          $key = rand (1000,9999);
$key = md5(key);

          это даст нам достаточно уникальный токен определенной длинны.
          так же нужно сделать таблицу
          token|action|exptime
          где
          token - собственно токен
          action - ссылка на действие
          exptime - время окончания действия токена (т. е. момент когда он протухнет)
          Ответить

          • ava Teddy_Brown 18.10.2011 13:03 # 0

            upd;
            ... но не зная контекста задачи - трудно говорить насколько это подходящее решение.
            Ответить

          • ava bot-minurast 18.10.2011 13:21 # 0

            md5( microtime(1) . rand(1000, 9999) )
            Ответить

            • ava Teddy_Brown 18.10.2011 13:28 # 0

              http://php.net/manual/ru/function.microtime.php

              есть небольшое ограничение по использованию.
              Ответить

              • ava bot-minurast 18.10.2011 13:37 # 0

                в любом случае приходится чем-то жертвовать
                Ответить

                • ava Teddy_Brown 18.10.2011 13:53 # 0

                  я бы увеличил интервал для рандома.
                  Ответить

                  • ava bot-minurast 18.10.2011 14:21 # 0

                    рандом - не торт.
                    тогда уж - uniqid
                    Ответить

                    • ava Teddy_Brown 18.10.2011 14:40 # 0

                      ты имеешь ввиду:
                      $key = md5(uniqid(rand(),1));

                      я правильно понял?
                      Ответить

                      • ava bot-minurast 18.10.2011 14:53 # 0

                        Ну типа того. Тогда уж наверняка "уникальный токен".
                        И "Когда новый пользователь переходит по ссылочке" - апдейтить значение этого хэша в нуль.
                        Ответить

                      • ava Lure Of Chaos 18.10.2011 14:54 # 0

                        md5(uniqid
                        БСК. md5 от uniqid только ухудшает энтропию
                        Ответить

                        • ava bot-minurast 18.10.2011 15:06 # 0

                          Насколько известно, существует всего несколько коллизий md5, которые можно пересчитать по пальцам.
                          Так что, навряд ли, ухудшает
                          Ответить

                          • ava Lure Of Chaos 18.10.2011 15:13 # 0

                            вы посчитайте кол-во вариантов md5 и кол-во вариантов uniqid .
                            в общем, брать хэш от случайного стринга не имеет смысла
                            Ответить

    • ava Nemoden 18.10.2011 08:58 # +4

      Автор как бы намекает на base64-md5-sha1 головного мозга.
      Ну это, в общем-то, бог с ним - каждый дрочит кодит как хочет, но зачем код подтерждения привязан на DBHOST, DBNAME?
      Ответить

      • ava bot-minurast 18.10.2011 09:19 # +4

        Просто там был ещё и DBPASSWORD, но начальство запротестовало... пришлось убрать
        Ответить

    • ava Teddy_Brown 18.10.2011 09:08 # 0

      просто автор can`t into tokens
      Ответить

    • ava nethak 18.10.2011 15:02 # −1

      На «гугл» с такой ссылкой бы не попал он) А в обще я бы на его месте проверял длину «GET», да и типичное «гавно», такой длины ссылка не к чему, т.к. у неё должен быть срок хранения в конце концов.
      Ответить

    • ava Hamer 24.10.2011 05:29 # 0

      Удивительная схожесть с кодом в DataLife Engine))))

      $_REQUEST['id'] = rawurlencode( base64_encode( $name . "||" . $email . "||" . md5( $password1 ) . "||" . md5( sha1( $name . $email . DBHOST . DBNAME . $config['key'] ) ) ) );
      Ответить

    Добавить комментарий