chihaya / chihaya

A customizable, multi-protocol BitTorrent Tracker
https://chihaya.io
Other
1.43k stars 190 forks source link

storage: add redis-socket scheme for connect to redis with unix socket #598

Open ru-asdx opened 1 year ago

ru-asdx commented 1 year ago

Ref: https://github.com/chihaya/chihaya/issues/595

address of redis storage now can be specified as :

redis_brocker: "redis-socket://pwd@/run/redis/redis.sock?db=0"

mrd0ll4r commented 1 year ago

Hello! The CI is fixed :) If you rebase this PR, we can probably merge it. Thanks again!

mrd0ll4r commented 1 year ago

Sorry, I missed this earlier: Can you squash your two commits into one and give it a name of this schema:

<subsystem, e.g. storage>: <what changed>

As described here in more detail.

Sorry for missing that last time! The changes look good.

ru-asdx commented 1 year ago

is something wrong again?

mrd0ll4r commented 1 year ago

Oh, sorry, this fell under the table. Changes look good, as before :) We have a data race which will probably block CI, again... really sorry.

Anyway, just leave this open, we'll fix it eventually and merge. Thank you!