From 356c7bec9c2900242b16d99968cdcd78e674dee9 Mon Sep 17 00:00:00 2001
From: Christian Schneppe <christian@pix-art.de>
Date: Sat, 24 Jun 2017 09:52:45 +0200
Subject: upgrade minidns. refactored dns utils

---
 .../de/pixart/messenger/xmpp/XmppConnection.java   | 39 ++++++++--------------
 1 file changed, 14 insertions(+), 25 deletions(-)

(limited to 'src/main/java/de/pixart/messenger/xmpp/XmppConnection.java')

diff --git a/src/main/java/de/pixart/messenger/xmpp/XmppConnection.java b/src/main/java/de/pixart/messenger/xmpp/XmppConnection.java
index f3ce20da3..a62f5c2d3 100644
--- a/src/main/java/de/pixart/messenger/xmpp/XmppConnection.java
+++ b/src/main/java/de/pixart/messenger/xmpp/XmppConnection.java
@@ -2,8 +2,6 @@ package de.pixart.messenger.xmpp;
 
 import android.graphics.Bitmap;
 import android.graphics.BitmapFactory;
-import android.os.Bundle;
-import android.os.Parcelable;
 import android.os.PowerManager;
 import android.os.PowerManager.WakeLock;
 import android.os.SystemClock;
@@ -20,7 +18,6 @@ import java.io.IOException;
 import java.io.InputStream;
 import java.math.BigInteger;
 import java.net.ConnectException;
-import java.net.IDN;
 import java.net.InetAddress;
 import java.net.InetSocketAddress;
 import java.net.Socket;
@@ -69,9 +66,10 @@ import de.pixart.messenger.generator.IqGenerator;
 import de.pixart.messenger.services.NotificationService;
 import de.pixart.messenger.services.XmppConnectionService;
 import de.pixart.messenger.ui.EditAccountActivity;
-import de.pixart.messenger.utils.DNSHelper;
+import de.pixart.messenger.utils.IP;
 import de.pixart.messenger.utils.Namespace;
 import de.pixart.messenger.utils.Patterns;
+import de.pixart.messenger.utils.Resolver;
 import de.pixart.messenger.utils.SSLSocketHelper;
 import de.pixart.messenger.utils.SocksSocketFactory;
 import de.pixart.messenger.xml.Element;
@@ -357,7 +355,7 @@ public class XmppConnection implements Runnable {
                 } catch (Exception e) {
                     throw new IOException(e.getMessage());
                 }
-            } else if (DNSHelper.isIp(account.getServer().toString())) {
+            } else if (IP.matches(account.getServer().toString())) {
                 localSocket = new Socket();
                 try {
                     localSocket.connect(new InetSocketAddress(account.getServer().toString(), 5222), Config.SOCKET_TIMEOUT * 1000);
@@ -373,37 +371,28 @@ public class XmppConnection implements Runnable {
                     throw new IOException(e.getMessage());
                 }
             } else {
-                final Bundle result = DNSHelper.getSRVRecord(account.getServer(), mXmppConnectionService);
-                final ArrayList<Parcelable> values = result.getParcelableArrayList("values");
-                for (Iterator<Parcelable> iterator = values.iterator(); iterator.hasNext(); ) {
+                List<Resolver.Result> results = Resolver.resolve(account.getJid().getDomainpart());
+                Log.d(Config.LOGTAG, "results: " + results);
+                for (Iterator<Resolver.Result> iterator = results.iterator(); iterator.hasNext(); ) {
+                    final Resolver.Result result = iterator.next();
                     if (Thread.currentThread().isInterrupted()) {
                         Log.d(Config.LOGTAG, account.getJid().toBareJid() + ": Thread was interrupted");
                         return;
                     }
-                    final Bundle namePort = (Bundle) iterator.next();
                     try {
-                        String srvRecordServer;
-                        try {
-                            srvRecordServer = IDN.toASCII(namePort.getString("name"));
-                        } catch (final IllegalArgumentException e) {
-                            // TODO: Handle me?`
-                            srvRecordServer = "";
-                        }
-                        final int srvRecordPort = namePort.getInt("port");
-                        final String srvIpServer = namePort.getString("ip");
                         // if tls is true, encryption is implied and must not be started
-                        features.encryptionEnabled = namePort.getBoolean("tls");
+                        features.encryptionEnabled = result.isDirectTls();
                         final InetSocketAddress addr;
-                        if (srvIpServer != null) {
-                            addr = new InetSocketAddress(srvIpServer, srvRecordPort);
+                        if (result.getIp() != null) {
+                            addr = new InetSocketAddress(result.getIp(), result.getPort());
                             Log.d(Config.LOGTAG, account.getJid().toBareJid().toString()
-                                    + ": using values from dns " + srvRecordServer
-                                    + "[" + srvIpServer + "]:" + srvRecordPort + " tls: " + features.encryptionEnabled);
+                                    + ": using values from dns " + result.getHostname().toString()
+                                    + "[" + result.getIp().toString() + "]:" + result.getPort() + " tls: " + features.encryptionEnabled);
                         } else {
-                            addr = new InetSocketAddress(srvRecordServer, srvRecordPort);
+                            addr = new InetSocketAddress(result.getHostname().toString(), result.getPort());
                             Log.d(Config.LOGTAG, account.getJid().toBareJid().toString()
                                     + ": using values from dns "
-                                    + srvRecordServer + ":" + srvRecordPort + " tls: " + features.encryptionEnabled);
+                                    + result.getHostname().toString() + ":" + result.getPort() + " tls: " + features.encryptionEnabled);
                         }
 
                         if (!features.encryptionEnabled) {
-- 
cgit v1.2.3