Android - implement setRegion about react-native-twilio-programmable-voice HOT 10 CLOSED

This is also an issue for iOS

from react-native-twilio-programmable-voice.

We sometimes encounter problems when our app runs behind:

• Mobile device management
• Custom APN
• Firewalled WiFi

I suppose I'm not the only one who has noticed these limits. The IP-ranges for all regions should be whitelisted by the firewall in order to get it to work.

The list of IPs is huge: https://www.twilio.com/docs/voice/client/regions

Forcing the connection through one region, makes the list of IPs which should be whitelisted smaller. This can be done in the TwilioVoice SDK according to the docs here: https://www.twilio.com/docs/voice/voip-sdk/regions

When the new SDK version lands, it might be interesting to consider implementing setRegion().

Let me know what your thoughts are.

from react-native-twilio-programmable-voice.

@jdegger @fabriziomoscon I believe we could have a set of constants referencing each of the new "edges" and allow users to set them using Voice.setEdge("ashburn-ix"); on android and TwilioVoice.edge = "ashburn-ix"; on iOS.
Something like what we have for connection status.
What do you think?

from react-native-twilio-programmable-voice.

Maybe just send the string value to setEdge/setRegion instead of adding the edges as contants, otherwise we would have to update our package for every new edge Twilio adds, right?

from react-native-twilio-programmable-voice.

Yes, that sounds just right. I'll try to send a PR later today with this.

from react-native-twilio-programmable-voice.

@jdegger sorry for my late reply, I've been using setEdge on iOS for a while locally so I moved on to implement it on android.
There's one catch anyway, since Android SDK uses setRegion till version 5.3.0 and setEdge from there on.
Should we implement the deprecated setRegion method or should we upgrade Android SDK to version 5.6.0 (Latest at the time of writing)?

from react-native-twilio-programmable-voice.

@fmonsalvo I think we should wait for @fabriziomoscon to finish his PR maybe? I think he chose 5.0.1 because he is updating step by step from the twilio docs.

I will discuss with him also.

from react-native-twilio-programmable-voice.

ps: thanks for your efforts, highly appreciated!

from react-native-twilio-programmable-voice.

Yes, let's wait for that PR to be merged and then I'll send mine.
I'll look into the other items in the backlog and see which one I can work on.

from react-native-twilio-programmable-voice.

This is superseded by https://www.twilio.com/docs/voice/voip-sdk/android/3x-changelog#530

from react-native-twilio-programmable-voice.