Race condition in Queue.Protect

[eGavr]

Selenoid version: 1.10.7
Docker version: 20.10.7, build f0df350
Selenoid config:

{"firefox":{"versions":{"95.0":{"image":"selenoid/firefox:95.0","path":"/wd/hub","port":"4444","tmpfs":{"/tmp":"size=128m"}}}}}

Selenoid run:

docker run -it --rm -p 4444:4444 -v /var/run/docker.sock:/var/run/docker.sock -v <DIR_WITH_CONFIG>:/etc/selenoid/:ro aerokube/selenoid:1.10.7 -disable-queue -limit 5

Test script:

for i in $(seq 1 10);
do
  curl --connect-timeout 30 --header "Content-Type: application/json" http://127.0.0.1:4444/wd/hub/session -d '{"desiredCapabilities":{"browserName":"firefox","browserVersion":"95.0"}}' & 
done

wait

Current behaviour (this behaviour is flaky, so you probably need run test script several times to reproduce the bug):

- less than 5 requests "immediately" fail with error "Queue is Full"
- Selenoid returns 5 browser sessions
- the remaining requests fail after 'curl' timeout exceeds

Expected behaviour:

- 5 requests "immediately" fail with error "Queue Is Full"
- Selenoid returns 5 browser sessions

---

JFYI. In our team we've hacked the problem this way.