redis lock expire fail

[ohroy]

• Laravel Version: 6.0.0
• PHP Version: 7.4.0
• Database Driver & Version:Redis 5

Description:

now logic to acquire a redis lock need 2 steps.
setnx and set expire .
I meet a bug that setnx is sucessed but set expire not. this may cause by network or other weird reasons, but I think this can fix by atomic operation
I fount a pr but it have't be accept, #29076

Steps To Reproduce:

1.breakpoint in

framework/src/Illuminate/Cache/RedisLock.php

Line 40 in dca4598

$this->redis->expire($this->name, $this->seconds);

2. try to acquire a redislock
3. wait the breakpoint fire, stop the redis server
4.resume the breakpoint and return
5. restore the redis server

then you will get a never expires lock...
Although the probability of its occurrence is very low, once it occurs, it will cause very serious consequences. In a real business scenario, I have been experiencing this problem recently. Just now I finally discovered the source of the problem.
I hope to be able to fix it.
Thank you.

[ChanJuiHuang]

Hi, Laravel.
Could you reopen this PR #29076 ?
I send this PR in advance!

[driesvints]

Should be fixed in the next release.