Skip to content

Commit

Permalink
fix: nodejs redis v4 interprets localhost as ipv6 address
Browse files Browse the repository at this point in the history
But the redis only binds to 127.0.0.1, not ::1.

Avoid this problem by being explicit.
  • Loading branch information
blaggacao committed Jul 1, 2023
1 parent 480e35c commit d366eb1
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion bench/config/common_site_config.py
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,7 @@ def update_config_for_frappe(config, bench_path):

for key in ("redis_cache", "redis_queue", "redis_socketio"):
if key not in config:
config[key] = f"redis://localhost:{ports[key]}"
config[key] = f"redis://127.0.0.1:{ports[key]}"

for key in ("webserver_port", "socketio_port", "file_watcher_port"):
if key not in config:
Expand Down

0 comments on commit d366eb1

Please sign in to comment.