Fix #14: сделано хранение состояния случайного генератора

fullstack-development / haskell-starter-kit

Starter Kit for web-backend apps written with Haskell, Servant, Docker and PostgreSQL

BSD 3-Clause "New" or "Revised" License

1 stars 1 forks source link

Fix #14: сделано хранение состояния случайного генератора #15

Closed antonkalinin-ml closed 3 years ago

antonkalinin-ml commented 3 years ago

добавил хранение стейта генератора. До этого использовался newStdGen, который делал split на глобальном генераторе, так что состояние генератора в целом сохранялось, но в последующем коммите используется криптографический генератор, который не умеет split.
упростил код genConfirmationCode. Генерация теперь прямолинейная, без дополнительного перемешивания.
приделал криптографический генератор ChaChaDRG из cryptonite, завернул его в IORef и приделал интерфейс RandomGen.

antonkalinin-ml commented 3 years ago

Стоп, тесты забыл починить :)

antonkalinin-ml commented 3 years ago

Я пока занят на другом проекте, могу не успеть доделать, хотя попробую. Если долго провисит, исправьте, пожалуйста, сами. stack test не билдится.

Znack commented 3 years ago

Да мы не торопимся, подождем, как время будет :)

antonkalinin-ml commented 3 years ago

Готово. Просьба поревьюить :)

antonkalinin-ml commented 3 years ago

Позже нашел функцию, которая решила бы все наши проблемы: https://hackage.haskell.org/package/random-1.2.0/docs/System-Random.html#v:getStdRandom . Но она привязана к StdGen, который не является криптографическим, так что я не буду удалять StatefulRandomGen.

antonkalinin-ml commented 3 years ago

@Znack смержи, пожалуйста, если замечаний не осталось и никто больше смотреть не будет :)

Znack commented 3 years ago

А я ещё хочу глянуть, но попозже, пока времени не могу найти :( Постараюсь сегодня

kelizarov commented 3 years ago

@antonkalinin-ml а можещь пожалуйста зарезолвить те дискашены, что были исправлены?

Znack commented 3 years ago

@kelizarov отревьюишь?

Znack commented 3 years ago

@antonkalinin-ml тут конфликты возникли :(

Znack commented 3 years ago

В общем чтобы сильно не затягивать, вмержу. @kelizarov при желании можешь тут откоментить в будущем :)