trigger.dev icon indicating copy to clipboard operation
trigger.dev copied to clipboard

fix: support for IPv6 redis connections

Open mittalyashu opened this issue 6 months ago • 1 comments

Closes #

✅ Checklist

  • [x] I have followed every step in the contributing guide
  • [ ] The PR title follows the convention.
  • [ ] I ran and tested the code works

Testing

While trying to self-host it, I was trying to add a Redis instance as result, I was getting this error in the logs.

[ioredis] Unhandled error event: Error: connect ETIMEDOUT
    at TLSSocket.<anonymous> (/triggerdotdev/node_modules/.pnpm/[email protected]/node_modules/ioredis/built/Redis.js:170:41)
    at Object.onceWrapper (node:events:631:28)
    at TLSSocket.emit (node:events:517:28)
    at Socket._onTimeout (node:net:598:8)
    at listOnTimeout (node:internal/timers:569:17)
    at process.processTimers (node:internal/timers:512:7)

Changelog

By default ioredis set the connection host to IPv4.

The documentation for the family parameter are: https://nodejs.org/docs/latest-v20.x/api/net.html#socketconnectoptions-connectlistener

mittalyashu avatar Aug 14 '24 12:08 mittalyashu