workerd icon indicating copy to clipboard operation
workerd copied to clipboard

Fix top level randomness in aiohttp

Open hoodmane opened this issue 1 year ago • 1 comments

aiohttp instantiates a random.Random() in order to be a default argument for WebSocketWriter. Instead, use the random module itself for this which implements the same interface as Random.

Should fix one of the test failures in #2096.

hoodmane avatar May 09 '24 19:05 hoodmane

It works:

2024-05-09T20:58:32.9101316Z //src/workerd/server/tests/python:import/aiohttp/import    
PASSED in 26.5s

hoodmane avatar May 09 '24 22:05 hoodmane

Merged as part of #2096 🥳

garrettgu10 avatar May 30 '24 19:05 garrettgu10