Support connection schemes: valkey://, valkeys://.
This patch try to:
- Support valkey:// and valkeys:// scheme, retain the original Redis scheme for compatibility.
- Add unit tests for valid URI.
Fixes: https://github.com/valkey-io/valkey/issues/198 Fixes: https://github.com/valkey-io/valkey/issues/200
AH!! I didn't review the tests. If there is more coming, better mark the PR as draft. :D
I'll review the tests now.
Please also update the top comment to describe the changes (don't just link issue). For example, mention we supports valkey://, and also retains the original redis schema for compatibility, and then we adds some tests to cover these cases.
also if you have a PR directly, there is actually no need to create a new issue (there is no discussion or tracking in the issue so it is actually worthless)
I'm going to merge this. It's the least surprising schemes for any forked clients and for users moving from redis-cli to valkey-cli.
We can change back if there are protests.