anycable / anycable-go

AnyCable real-time server
https://anycable.io
MIT License
364 stars 65 forks source link

Add support for REDIS_URL with trailing slash hostnames #190

Closed ardecvz closed 1 year ago

ardecvz commented 1 year ago

What is the purpose of this pull request?

Add support for REDIS_URL with trailing slash hostnames (redis://localhost:6379/)

What changes did you make? (overview)

Chomp trailing slash in the hostname

Is there anything you'd like reviewers to focus on?

Upstream this change to rueidis URL parsing as the implementation doesn't tolerate the trailing slash hostnames (redis-cli does).

Checklist