<div dir="ltr">Ok,<div><br></div><div style>I'll adapt my patch to make the avpf=yes/no flag fully ignored for inbound calls.</div><div style><br></div><div style>Thanks for your feedback!</div><div style><br></div><div style>
Torrey</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 3 July 2013 09:33, Olle E. Johansson <span dir="ltr"><<a href="mailto:oej@edvina.net" target="_blank">oej@edvina.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
3 jul 2013 kl. 09:21 skrev Torrey Searle <<a href="mailto:tsearle@gmail.com">tsearle@gmail.com</a>>:<br>
<div><div class="h5"><br>
> Hello,<br>
><br>
> I've found that the current behavior of ast11 is as follows:<br>
><br>
> 1. if avpf=no all avpf calls get rejected<br>
> 2. if avpf=yes all avp calls get rejected<br>
><br>
> However by comparison I following behavior with regards to encryption<br>
> 1. if encyrption=no both normal and srtp calls are accepted<br>
> 2. if encryption=yes unencrypted calls are rejected<br>
><br>
> I have a patch to adapt the avpf behavior to match the encryption behavior (if avpf=no enable avpf if it's in the offer). I was wondering if people would be happy if I submit the patch as-is or would prefer a different way of making avpf optional?<br>
<br>
</div></div>I think the option should indicate what we send, not what we can accept when we receive. If you want that, maybe another setting is needed to say "I only accept calls with AVPF" but I don't see a reason for that.<br>
<br>
I do see a reason for that option in regards to encryption though.<br>
<br>
/O<br>
<br>
<br>
--<br>
_____________________________________________________________________<br>
-- Bandwidth and Colocation Provided by <a href="http://www.api-digital.com" target="_blank">http://www.api-digital.com</a> --<br>
<br>
asterisk-dev mailing list<br>
To UNSUBSCRIBE or update options visit:<br>
<a href="http://lists.digium.com/mailman/listinfo/asterisk-dev" target="_blank">http://lists.digium.com/mailman/listinfo/asterisk-dev</a><br>
</blockquote></div><br></div>