diff options
author | Christian Schneppe <christian@pix-art.de> | 2017-06-24 09:52:45 +0200 |
---|---|---|
committer | Christian Schneppe <christian@pix-art.de> | 2017-06-24 09:52:45 +0200 |
commit | 356c7bec9c2900242b16d99968cdcd78e674dee9 (patch) | |
tree | c44021992d825db96fcc4f84400cd63cc8d16450 /src/main/java/de/pixart/messenger/xmpp/XmppConnection.java | |
parent | 8e897c3d69d65155a2fa7a4ce75bfc5bc13d7233 (diff) |
upgrade minidns. refactored dns utils
Diffstat (limited to '')
-rw-r--r-- | src/main/java/de/pixart/messenger/xmpp/XmppConnection.java | 39 |
1 files changed, 14 insertions, 25 deletions
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) { |