net.resolvers.basic: Disable DANE for now, completely broken

Turns out 'extra' is, at least for mod_s2s, the same table for *all*
connections.

CHANGES

@@ -17,7 +17,6 @@ TRUNK
 -   `daemonize` option deprecated
 -   SASL DIGEST-MD5 removed
 -   Switch to libunbound for DNS queries
--   DANE Support
 -   mod_external_services (XEP-0215)
 -   util.error for encapsulating errors
 -   MUC: support for XEP-0421 occupant identifiers

net/resolvers/basic.lua

@@ -36,15 +36,14 @@ function methods:next(cb)
 		n = n - 1;
 		if n > 0 then return; end
 		self.targets = targets;
+		--[[
+		-- TODO stash tlsa somewhere per connection
+		-- FIXME 'extra' here is not per connection
 		if self.extra and self.extra.use_dane then
 			if secure and tlsa[1] then
-				self.extra.tlsa = tlsa;
-				self.extra.dane_hostname = self.hostname;
-			else
-				self.extra.tlsa = nil;
-				self.extra.dane_hostname = nil;
 			end
 		end
+		--]]
 		self:next(cb);
 	end