Don't include IPv4 or Docker addresses in connection config sent via ADB

This fixes a bug where devices would connect via IPv6, which isn't
actually supported. This worked with QR codes (because the UI does
correctly do this filtering) but not with ADB because of this issue. It
also reduces unnecessary network checks (and waiting for timeouts,
potentially) for some cases too.

Author: pimterry

src/interceptors/android/android-adb-interceptor.ts

@@ -103,9 +103,15 @@ export class AndroidAdbInterceptor implements Interceptor {
                 '10.0.3.2', // Genymotion localhost ip
             ].concat(
                 // Every other external network ip
-                _.flatMap(os.networkInterfaces(), (addresses) =>
+                _.flatMap(os.networkInterfaces(), (addresses, iface) =>
                     (addresses || [])
-                        .filter(a => !a.internal)
+                        .filter(a =>
+                            !a.internal && // Loopback interfaces
+                            a.family === "IPv4" && // Android VPN app supports IPv4 only
+                            iface !== 'docker0' && // Docker default bridge interface
+                            !iface.startsWith('br-') && // More docker bridge interfaces
+                            !iface.startsWith('veth') // Virtual interfaces for each docker container
+                        )
                         .map(a => a.address)
                 )
             ),