ed-org-ua / nabuvote Goto Github PK

E-Voting System for Public Control Council

PHP 95.83% CSS 1.55% JavaScript 2.62%

nabuvote's Introduction

National Anti-Corruption Bureau of Ukraine
E-Voting System for Public Control Council
Complies TU U 58.29.4-39582446-001:2015 v1
Copyright NGO E-DEMOCRACY http://ed.org.ua

For further information, please contact us
via e-mail <[email protected]>

Next text in Russian only in UTF-8 charset
try use translate.google.com if necessary.

Система рейтингового интернет голосования

Появилась где-то между 5 и 10 часами вечера 26 мая 2015 года, как альтернатива существующей тогда системе выборов Общественного совета при Национальном антикоррупционном бюро Украины (http://nabu.gov.ua).

Используемые проверки и ограничения

Для предотвращения автоматического и повторного голосования в системе используются следующие проверки:

Анти-робот тест (используется Google reCAPTCHA)
Проверка адреса электронной почты с вводом кода проверки из письма
Проверка номера мобильного телефона с получением кода в SMS

Пользовательские ограничения

Общее ограничение на время одной сессии: от теста reCAPTCHA до выбора кандидатов не более 15 минут.
Ограничение на общее количество POST запросов в рамках одной сессии.
Ограничение на количество попыток ввода кодов проверки e-mail и номера мобильного телефона.
Номер телефона и адрес электронной почты можно использовать только для одного голосования.

Открытый файл протокола голосования

Частично анонимизированные данные проголосовавших с указанием выбранных кандидатов доступны в режиме on-line.

Принципы разработки

Невероятно сжатые сроки разработки и необходимость прохождения аудита безопасности для использования в государственных органах привели к выбору следующих принципов построения системы:

Отказ от универсальных решений
Решение только задачи выборов Общественного совета НАБ Украины
Минимальное количество кода необходимого для аудита
Минимальное использование дополнительных библиотек
Отказ от использования фреймворков на стороне сервера
Поддержка мобильных устройств и адаптивная верстка
Защита от известных XSS и CSRF атак
Защита от инъекций в пользовательских данных
Подключение к базе данных только при необходимости

При отправке замечаний к коду пожалуйста учитывайте эти требования.

Системные требования

PHP >= 5.3
MySQL >= 4.1
php-mysqli
php-curl (для sms-gate)

Установка

Распакуйте архив в папку доступную веб-серверу
Создайте или подготовьте базу данных MySQL
Переименуйте settings.example.php в settings.php
Заполните недостающие значения в settings.php
Запустите из командной строки php install.php
При необходимости отредактируйте candidates.php
После завершения установки удалите install.php

Аудит исходного кода

Передана на базовый аудит в Государственную службу специальной связи и защиты информации Украины. Результаты аудита не известны.

Аудит исходного кода приветствуется. Замечания присылайте на адрес указанный ниже. Пожалуйста учитывайте принципы и ограничения описанные выше - это не универсальная система.

Лицензия

В рамках Меморандума о сотрудничестве передана для использования в выборах Общественного совета при Национальном антикоррупционном бюро Украины.

Для использования в других официальных выборах обращайтесь по контактам приведенным ниже.

Контакты

Общественная организация "Электронная демократия"

Сайт: http://ed.org.ua/

E-mail: [email protected]

Также см. раздел контакты на сайте.

nabuvote's People

Contributors

Stargazers

Watchers

Forkers

umkus vanuan a-b johnfelipe lobey86 hennz

nabuvote's Issues

Move out of DOCUMENT_ROOT all except of static/* and pages

It's more reliable then dancing with .htaccess

Remove always undeclared variable checking

https://github.com/imaginal/nabuvote/blob/master/system/functions.php#L558
https://github.com/imaginal/nabuvote/blob/master/system/functions.php#L575
https://github.com/imaginal/nabuvote/blob/master/system/functions.php#L591
$candidates is undefined here and can't be not empty

повысить шансы email нотификаций не попадать в спам

Здравствуйте, увидел пост на фб

https://www.facebook.com/anatoly.chernenko.25/posts/916594458363866

Заранее прошу простить, если не пригодится.

перед строкой https://github.com/imaginal/nabuvote/blob/master/system/functions.php#L296

чтобы не попадать в спам листы желательно добавить

"Return-Path: {$settings['email_from_header']}\r\n" .

и если позволит аудит, так же

в строке https://github.com/imaginal/nabuvote/blob/master/system/functions.php#L307

изменить на

mail($email, $subject, $message, $headers, "-f {$settings['email_from_header']}");

описано тут
http://us2.php.net/manual/en/function.mail.php

в параграфе "additional_parameters"

не рабоает если включен safe_mode, хотя в последних версиях этот режим deprecated

Вероятная ошибка в ссылках на архивы

https://github.com/imaginal/nabuvote/blob/master/system/candidates.php#L43
В конфигурации кандидатов, начиная с id=40, ссылки на архивы не соответствуют id кандидата

Something modern?

http://www.codeigniter.com/

Feature. browser-uniqueness

Привет,
может быть, это будет полезным для следующих версий - panopticlick.eff.org. У них на сайте есть статья про устойчивую идентификацию браузера посетителя сайта.

check_csrf_token improvement

В коде

function check_csrf_token() {
if (empty($_SESSION['csrf_token']))
return false;
if ($_POST['csrf_token'] != $_SESSION['csrf_token'])
die("csrf protection");
unset($_SESSION['csrf_token']);
}

перенести
unset($_SESSION['csrf_token']);
перед die. Иначе при вызове die() до этой строки не дойдет, и в сессии остается одно и то же значение, увеличивая с каждым запросом мизерную (с учетом капчи), но все же вероятность подбора csrf_token

В требованиях указать используемый вебсервер

В коде есть htaccess, но в требованиях нет apache.
Нужно или добавить варианты конфига для nginx, или четко прописать, что нужен именно apache.

Test coverage

Do we have any ideas how to test this code?

Remove workaround with seed

https://github.com/imaginal/nabuvote/blob/master/system/functions.php#L171

Why it would be empty?
If we assume that it's good random data (and environment allows using of this openssl PHP function), why don't use it directly instead of rand (limits may be implemented with mod)? I see that now we call seeding every time before we call rand().
Why use machine-depending 'i' option for unpack if we know that length is 4 bytes and know that 'L' is always 4 bytes too?

P.S. also as in PHP "empty" returns true for "", 0 and ever "0", it's bad practice to use it in production in such cases.
It may work but when someone reads $a === '' he knows that you mean that $a may be empty string.
When someone reads empty($a) he don't knows, is $a may be false, '', 0, array() or even '0'.

Сделайте результаты голосование скрытыми до конца голосования

#15 - то же самое что и тут, только наоборот.

Новая пара private/public key.
результаты персонального голосование шифруете публичным ключем + random строчка вконце.
В конце голосование публикуете private key, ну и розшифровываете все результаты.

Это гарантирует что результаты голосование нельзя манипулировать и они будут тайными до конца голосования.

Ну тут сложнее так как нам прийдеться доверять сисадмину что он приватный ключ никому не покажет до конца голосования.

Проверка правильности e-mail'а делается два раза

https://github.com/imaginal/nabuvote/blob/master/step2.php#L47
https://github.com/imaginal/nabuvote/blob/master/system/functions.php#L291

К тому же, как-то мало условий валидаций, но я пока не нашёл конкретного контр-примера. (php реализация mail)

can you share screenshots?

Подпись результатов с помощью openssl_private_encrypt

https://nabu-test.ed.org.ua/public/report.txt - возвращает результаты какие то во время голосование.
Потом я скажу что мне показывались результаты другие во время голосование, и сделаю скрин какой то. И буду говорить что вы врете.
Что бы этого не случилось, подпишите результаты приватным ключем, и опубликуйте публичный ключ, что бы можна было проверить результаты.

Процедура следующая (по коду это где то 100 строчек, смотрите пример):

Генерируете private/public ключи.
Публикуете public ключ.
Все результаты персональные и report.txt подписываете, private ключем.
Каждый человек может проверить это с помощью публичного ключа.
В конце голосование удаляете private ключ.
Если кто нибуть показывает вам результаты голосование которые не соответсвуют опубликованым вами в конце и можна проверить public key.
То это значит что ваша система не работает, кто то хакнул сервер или сисадмин подхимичил что то, или кто то украл private key, что тоже самое что кто то вас хакнул и результаты не действительные.

http://www.programering.com/a/MDO2cDMwATQ.html - это достаточно просто, порядка 100 строчек