Re: [PATCH 1/6] maintenance: add get_random_minute()

From

Derrick Stolee <derrickstolee@github.com>

Date

Aug 9, 2023, 12:17 UTC

Message-ID

<a1e7d730-e220-48ec-8393-2d3538b80163@github.com>

In-Reply-To

<ZNKfxOWJAuJ5DxAN@nand.local>

On 8/8/2023 4:04 PM, Taylor Blau wrote:

> On Tue, Aug 08, 2023 at 01:28:50PM -0400, Derrick Stolee wrote:

>> I see you're moving ahead with removing the srand() from the lockfile code,

> That thread may have progressed a little since you last looked at it.

I think this part of my summary is still correct.

>> so I'll focus on creating a `git_rand()` that centralizes the use of
>> srand(), but won't touch the code in the lockfile so your patch applies
>> independently.

> Instead of using srand() and rand() (which would make sense to wrap with
> git_rand() as you propose), we can simplify our lives by using a CSPRNG,
> which only gets initialized once, as is already the case with
> csprng_bytes().

So the idea is to use csprng_bytes() everywhere instead of srand()/rand().