7 ms·
You criticized about the random code generation function. Could you explain why it is bad? Though I code, I am no expert and would really like to know.
by system_32 12y ago
You criticized about the random code generation function. Could you explain why it is bad? Though I code, I am no expert and would really like to know.
- sarciszewski 12y agoThis is the offending code: https://github.com/bmmayer/ephchat/blob/ec375c7974ea825f887ff4ca1cca2349018a187e/php/functions.php#L4-L20 https://github.com/bmmayer/ephchat/blob/ec375c7974ea825f887f... This is the proper way to do it (h/t @tptacek): http://sockpuppet.org/blog/2014/02/25/safely-generate-random-numbers/ http://sockpuppet.org/blog/2014/02/25/safely-generate-random... mt_srand() + rand() is just hilarious. The md5(uniquid()) thing is a common randomness anti-pattern in PHP projects that needs to die in a fire. In PHP, a very brief example of the code to achieve the proper way of generating randomness looks like the snippet I posted in the issue.