Hi,
I'll check the links you provided and commit the fix if appropriate. Thanks a
lot for the detailed report!
Regards,
On Feb 27, 2012, at 12:45 PM, Nikita Kozlov wrote:
> Hello,
>
> I think I have found a bug in media-relay if it's installed on a host without
> any name resolver (empty /etc/resolv.conf, only /etc/hosts).
>
> If mediaproxy is used with twitsted 10.X, the twisted defers' will fail and
> we will enter in the _eb_no_srv callback but with a failure type of
> TimeoutError.
> Since TimeoutError isn't "traped" in _eb_no_srv we are not trying to resolve
> it further because we are not executing reactor.resolve().
>
> Feb 21 15:15:32 testnikiproxy2 media-relay[1086]: error: Unhandled error in
> Deferred:
> Feb 21 15:15:32 testnikiproxy2 media-relay[1086]: TimeoutError:
> [Query('_sip._udp.testnikisip2', 33, 1)]
>
> Note that I'm not sure if the error will happen with twisted 11.0+ since the
> method setTimeout and TimeoutError were deprecated...
> http://twistedmatrix.com/trac/browser/tags/releases/twisted-10.0.0/twisted/internet/defer.py#L399
>
> http://twistedmatrix.com/trac/browser/tags/releases/twisted-11.0.0/twisted/internet/defer.py#L399
> (setTimeout is not longer defined)
>
> Anyway, here is a patch to fix it for twisted 10.0 :
>
> diff -rN -u old-mediaproxy/mediaproxy/relay.py
> new-mediaproxy/mediaproxy/relay.py
> --- old-mediaproxy/mediaproxy/relay.py 2012-02-27 12:07:58.100162109 +0100
> +++ new-mediaproxy/mediaproxy/relay.py 2012-02-27 12:07:58.123162912 +0100
> @@ -16,7 +16,7 @@
> from twisted.protocols.basic import LineOnlyReceiver
> from twisted.internet.error import ConnectionDone, TCPTimedOutError,
> DNSLookupError
> from twisted.internet.protocol import ClientFactory
> -from twisted.internet.defer import DeferredList, succeed
> +from twisted.internet.defer import DeferredList, succeed, TimeoutError
> from twisted.internet import reactor
> from twisted.python import failure
> from twisted.names import dns
> @@ -260,7 +260,7 @@
> raise DomainError
>
> def _eb_no_srv(self, failure, addr, port):
> - failure.trap(DomainError)
> + failure.trap(DomainError, TimeoutError)
> return reactor.resolve(addr).addCallback(lambda host: (host,
> port)).addErrback(self._eb_no_dns, addr)
>
> def _eb_no_dns(self, failure, addr):
>
>
> Regards,
> --
> Nikita
>
> _______________________________________________
> Users mailing list
> [email protected]
> http://lists.opensips.org/cgi-bin/mailman/listinfo/users
--
Saúl Ibarra Corretgé
AG Projects
_______________________________________________
Users mailing list
[email protected]
http://lists.opensips.org/cgi-bin/mailman/listinfo/users
