<div dir="ltr"><div>Hi Madaro,<br></div><div><br></div><div>Thanks for the patch. I finally reviewed it and it looks good to me. I've just pushed the patch to my github repo.</div><div><br></div><div>Thanks, and sorry for the delay!</div>
<div>Youness.</div><div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jan 20, 2014 at 5:33 AM, Madaro Livio <span dir="ltr"><<a href="mailto:livio.madaro@telecomitalia.it" target="_blank">livio.madaro@telecomitalia.it</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello,<br>
I think this patch is still pending. Has it been rejected or is it under review?<br>
<br>
Regards,<br>
Livio<br>
<br>
<br>
-----Original Message-----<br>
From: nice-bounces+livio.madaro=<a href="mailto:telecomitalia.it@lists.freedesktop.org">telecomitalia.it@lists.freedesktop.org</a> [mailto:<a href="mailto:nice-bounces%2Blivio.madaro">nice-bounces+livio.madaro</a>=<a href="mailto:telecomitalia.it@lists.freedesktop.org">telecomitalia.it@lists.freedesktop.org</a>] On Behalf Of Madaro Livio<br>

Sent: Thursday, March 21, 2013 11:02 AM<br>
To: <a href="mailto:nice@lists.freedesktop.org">nice@lists.freedesktop.org</a><br>
Subject: Re: [Nice] patch for TURN nonce expired<br>
<br>
Hi,<br>
I think the error is not specific to TURN because it is defined in RFC5389. I think the nice agent can already handle the  error 438, but it is "filtered" by the stun agent.<br>
<br>
Regards,<br>
Livio<br>
<br>
-----Original Message-----<br>
From: nice-bounces+livio.madaro=<a href="mailto:telecomitalia.it@lists.freedesktop.org">telecomitalia.it@lists.freedesktop.org</a> [mailto:<a href="mailto:nice-bounces%2Blivio.madaro">nice-bounces+livio.madaro</a>=<a href="mailto:telecomitalia.it@lists.freedesktop.org">telecomitalia.it@lists.freedesktop.org</a>] On Behalf Of Youness Alaoui<br>

Sent: Wednesday, March 20, 2013 8:50 PM<br>
To: <a href="mailto:nice@lists.freedesktop.org">nice@lists.freedesktop.org</a><br>
Subject: Re: [Nice] patch for TURN nonce expired<br>
<br>
Humm.. I'm not sure this is done the right way. Shouldn't the error be validated in the nice agent rather than stun agent, since that error is specific to turn, right ?<br>
I think libnice should detect that error and clear the nonce and retry, instead of considering that the message is not authenticated.<br>
I'll have to think about it, but I'm a bit busy now.<br>
<br>
Thanks,<br>
Youness.<br>
<br>
On 02/28/2013 05:48 AM, Madaro Livio wrote:<br>
> Hi,<br>
><br>
> I think there is a problem when  TURN nonce expires.<br>
><br>
> If the nonce expires the TURN server replies with error code 438 when<br>
> libnice sends a TURN allocation refresh.  libnice should re-send the<br>
> allocation refresh with the new nonce but libnice fails to validate<br>
> the error code 438 (STUN_VALIDATION_UNAUTHORIZED).<br>
><br>
> Using <a href="http://turnserver.org" target="_blank">turnserver.org</a> the nonce expires after 1 hour.<br>
><br>
><br>
><br>
> I send a patch for review. I just added a check for error code 438 in<br>
> stun validation (stun_agent_validate). Do you think the check for<br>
> error 438 is missing elsewhere?<br>
><br>
><br>
><br>
> Regards,<br>
><br>
> Livio<br>
><br>
><br>
><br>
> Questo messaggio e i suoi allegati sono indirizzati esclusivamente<br>
> alle persone indicate. La diffusione, copia o qualsiasi altra azione<br>
> derivante dalla conoscenza di queste informazioni sono rigorosamente<br>
> vietate. Qualora abbiate ricevuto questo documento per errore siete<br>
> cortesemente pregati di darne immediata comunicazione al mittente e di provvedere alla sua distruzione, Grazie.<br>
><br>
> /This e-mail and any attachments// is //confidential and may contain<br>
> privileged information intended for the addressee(s) only.<br>
> Dissemination, copying, printing or use by anybody else is<br>
> unauthorised. If you are not the intended recipient, please delete<br>
> this message and any attachments and advise the sender by return<br>
> e-mail, Thanks./<br>
><br>
> *rispetta l'ambienteRispetta l'ambiente. Non stampare questa mail se<br>
> non è<br>
> necessario.*<br>
><br>
><br>
><br>
> _______________________________________________<br>
> nice mailing list<br>
> <a href="mailto:nice@lists.freedesktop.org">nice@lists.freedesktop.org</a><br>
> <a href="http://lists.freedesktop.org/mailman/listinfo/nice" target="_blank">http://lists.freedesktop.org/mailman/listinfo/nice</a><br>
><br>
<br>
<br>
<br>
Questo messaggio e i suoi allegati sono indirizzati esclusivamente alle persone indicate. La diffusione, copia o qualsiasi altra azione derivante dalla conoscenza di queste informazioni sono rigorosamente vietate. Qualora abbiate ricevuto questo documento per errore siete cortesemente pregati di darne immediata comunicazione al mittente e di provvedere alla sua distruzione, Grazie.<br>

<br>
This e-mail and any attachments is confidential and may contain privileged information intended for the addressee(s) only. Dissemination, copying, printing or use by anybody else is unauthorised. If you are not the intended recipient, please delete this message and any attachments and advise the sender by return e-mail, Thanks.<br>

<br>
_______________________________________________<br>
nice mailing list<br>
<a href="mailto:nice@lists.freedesktop.org">nice@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/nice" target="_blank">http://lists.freedesktop.org/mailman/listinfo/nice</a><br>
_______________________________________________<br>
nice mailing list<br>
<a href="mailto:nice@lists.freedesktop.org">nice@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/nice" target="_blank">http://lists.freedesktop.org/mailman/listinfo/nice</a><br>
</blockquote></div><br></div>