matrix-org / matrix-appservice-irc

Node.js IRC bridge for Matrix
Apache License 2.0
460 stars 151 forks source link

Timeout waiting for ping event delays bridge startup (Ping is disabled on homeserver by choice) #1621

Open FlyveHest opened 1 year ago

FlyveHest commented 1 year ago

Describe the bug

To Reproduce Steps to reproduce the behavior:

  1. Have ping disabled on Homeserver
  2. Start IRC bridge
  3. Wait for ping to timeout before bridge starts

Expected behavior Bridge starting up without running a ping test

Desktop (please complete the following information):

Additional context It seems like a pingtest was added, and ping is disabled on my homeserver (by choice), which causes the bridge startup to hang until the ping attempt times out.

There are no connectivity issues between the bridge and the homeserver, so it would be nice to at least have the possibility to disable this naive connection check on startup. (Its fine enough to let it be enabled by default)