search

jcasbin / redis-watcher

Redis watcher for jCasbin
https://github.com/casbin/jcasbin
Apache License 2.0
4 stars 12 forks source link

Add connect redis with password #11

Closed rongfengliang closed 3 years ago

rongfengliang commented 3 years ago

default only contains IP address and port does not include the password. this may be not safe in production

casbin-bot commented 3 years ago

@tangyang9464 @shy1st @fangzhengjin

hsluoyz commented 3 years ago

@rongfengliang can you make a PR to add it?

rongfengliang commented 3 years ago

@rongfengliang can you make a PR to add it?

ok. I will add it

hsluoyz commented 3 years ago

Or @seriouszyx can you try it?

rongfengliang commented 3 years ago

@hsluoyz I will do it, I have add this feature and also add ignore self message （our project ). but currently I'm too busy, Wait for a period of time I will merge it into this project

hsluoyz commented 3 years ago

@rongfengliang OK, looking forward to it :)

hsluoyz commented 3 years ago

@elfisworking @seriouszyx can anyone take this?