Redis dependency discrepancy about channels_redis

[marksweb]

It means Redis itself not redis-py. It was updated when we used some commands introduced in Redis 5.

There is no redis-py 5.

Yes, confusion on my part with you mentioning "redis-py" because of the package py-redis and too many days looking at pypi & packages as part of this upgrade job also likely playing a part.

If you can reduce these to something here, happy to take a look — It's not 100% clear yet that either redis-py's new async stuff is fully baked, or that we're wrapping it just right, so there could well be issues.

At the moment I don't think there's a problem with this pacakge. I've noticed the asgi side of the project isn't using ProtocolTypeRouter from channels itself yet. So I'm going to continue to hunt down the issue moving through the v4 release notes on there.

Thanks @carltongibson!

[carltongibson]

Cheer Mark. Yes, likely I'm getting it backwards.
🤯

I'm going to have to invoke the You know what he means school. 😝

[carltongibson]

It means Redis itself not redis-py. It was updated when we used some commands introduced in Redis 5.

There is no redis-py 5.

[carltongibson]

I've got web sockets failing

If you can reduce these to something here, happy to take a look — It's not 100% clear yet that either redis-py's new async stuff is fully baked, or that we're wrapping it just right, so there could well be issues.