onAutoreconnectIteration issues

[Kain Bryan-Jones]

We have an app written and we have our own onConnect handling. We have the following code...

We wanted to write custom logic for auto-reconnection, but we found that Tinode has its

own so we made a suboptimal hack with the above code which overrides the iteration timeout.

This has worked pretty much always until recently we had an issue. We had some network fluctuations and encountered the following log "onAutoreconnectIteration will retry in 4774.516907270728 ms"
and it never retries. I have looked into the source code and can't find a reason why onAutoreconnectIteration would just stop?
Can somebody help with this? Is there a max retry? Also is there a better way to handle this? Thank you.

[Gene]

You can change this line https://github.com/tinode/tinode-js/blob/47299d3f8ee0cb98c3f09489d6de6be704eed62b/src/connection.js#L26 to 

const _BOFF_MAX_ITER = 2;

Then you can remove your code.

[Gene]

BTW, unless you are dealing with automatic messaging (non-interactive), then you should not set it to 8 seconds. It's going to eat the battery on mobile.

[Kain Bryan-Jones]

So your recommendation is to clone the project to change this _BOFF_MAX_ITER? 

I will make a request on GitHub I feel like this should be configurable via the Tinode class.

And yes we are aware 8 seconds is short, but our application is being used in emergency situations on where we need reconnection  to be fast.

[Gene]

Sure, send a PR.