Misskey: AIDのノイズがあまりよくない

[mei23]

Summary

LinuxのPIDがほぼ連番に近い＋worker.idも1とか2 なので
ほぼ同じような値からのカウンタ値になっちゃいそう、乱数でいいかも
https://github.com/syuilo/misskey/blob/9c062643e17cc10fa9a5ff8cea2de27c62a1f695/src/misc/id/aid.ts#L7

[syuilo]

ノイズはほぼ同じ時刻で生成されたときにかぶらないようにする目的ですが、(時間ベースの)乱数にすると、ほぼ同じ時刻で生成すると乱数も同じになるのではないかという懸念がありました

[syuilo]

前に主要なMath.randomの実装はシードが時間ベースだという記事を読んだ気がしましたが、今ググっても特に出てこないので大丈夫かも

[rinsuki]

crypto.randomBytesとか使えば良いのでは

[mei23]

ほぼ同じ時刻で生成すると乱数も同じになるのではないか

ちょっと調べたけど結局よくわからなかった
ただuuidもulidもcrypto使ってる、cryptoは速度どうなのだろう

[syuilo]

とりあえずカウンター初期値をn倍するとかで今より多少マシになるかも？

[mei23]

他のサーバーがあって同じようなPIDが生成された場合もあって
めんどくさいので最初からcryptoでカウンタ作っちゃうでいいかも