You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The matrix bot sdk seems to only try connecting to the server once, and doesn't retry, so if the bridges and synapse are started at the same time (e.g. in docker swarm, which expects applications to handle retry logic themselves) then the bridge just errors instantly from not being able to resolve the hostname, leaving it to the downstream consumer to properly handle it.
Proposed solution
I took a look and it doesn't seem like the request library this uses has a built-in way to do retries, so this probably goes along with #90. If got is chosen, that has nice facilities for handling retries: https://github.com/sindresorhus/got#retry
Alternatives
It could make sense to leave it to the downstream consumer to properly handle this stuff, but I think it would be better if this were handled upstream with the goal of keeping the matrix bot stuff as easy to use as possible. I don't think there is any real harm in retrying at least a few times for a few minutes.
The text was updated successfully, but these errors were encountered:
The problem to solve
The matrix bot sdk seems to only try connecting to the server once, and doesn't retry, so if the bridges and synapse are started at the same time (e.g. in docker swarm, which expects applications to handle retry logic themselves) then the bridge just errors instantly from not being able to resolve the hostname, leaving it to the downstream consumer to properly handle it.
Proposed solution
I took a look and it doesn't seem like the request library this uses has a built-in way to do retries, so this probably goes along with #90. If got is chosen, that has nice facilities for handling retries: https://github.com/sindresorhus/got#retry
Alternatives
It could make sense to leave it to the downstream consumer to properly handle this stuff, but I think it would be better if this were handled upstream with the goal of keeping the matrix bot stuff as easy to use as possible. I don't think there is any real harm in retrying at least a few times for a few minutes.
The text was updated successfully, but these errors were encountered: