<div dir="ltr">Hi Carsten,<div><br></div><div>Thanks for your responce and please excuse my late reply too. I'm still working on the changes and will make a pull request as soon as I am ready. It will be a separate module which handles the IPSec tunnel creation/tear down, so that <span style="font-size:12.8px">ims_register_pcscf won't be polluted with platform specific functionality</span>. You are right, that new module can be ifdef-ed and replaced with something *BSD specific or whatever OS someone wants to use.</div><div><br></div><div>Best regards,</div><div>Tsvetomir</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Oct 13, 2017 at 6:21 AM, Carsten Bock <span dir="ltr"><<a href="mailto:carsten@ng-voice.com" target="_blank">carsten@ng-voice.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Tsvetomir,<br>
<br>
sorry for the late reply. I assume this mail got lost a bit in the<br>
days of Astricon. I even asked Daniel about this mail during Astricon,<br>
but he hadn't seen it yet. Right now, I'm officially on holiday....<br>
<br>
Can you please provide a Pull-Request for the changes?<br>
<br>
>From my perspective, it is likely fine to have a Linux-Only module, it<br>
might not be the first one. If you can encapsulate your extensions<br>
with some IFDEF's, so the functionality can be disabled on non-Linux,<br>
then that would be fine with me.<br>
<br>
It would be great, if Daniel or anyone else from the Management-Group<br>
could answer or comment this one as well??<br>
<br>
Thanks,<br>
Carsten<br>
<div><div class="h5"><br>
2017-10-04 10:14 GMT-04:00 Tsvetomir Dimitrov <<a href="mailto:tsv.dimitrov@gmail.com">tsv.dimitrov@gmail.com</a>>:<br>
> Hello,<br>
><br>
> I am working on a functionality which handles ipsec tunel creation for VoLTE<br>
> registration and I'd like to contribute it to the project. However the code<br>
> is heavily Linux specific - uses xfrm framework, so it won't compile on<br>
> distribution with older kernels and definitely won't compile on *BSD.<br>
><br>
> How problematic is this? How to handle this implementation so that it gets<br>
> merged?<br>
><br>
> Right now I can see two options:<br>
> 1. Implement the functionality in ims_register_pcscf.<br>
> 2. Implement separate ipsec module and handle the tunel creation/tear down<br>
> from the configuration.<br>
><br>
> The first solution is definitely the easiest one for implementation, but<br>
> after my patch the module won't be as portable as it is now and I'm afraid<br>
> my patch will be rejected.<br>
><br>
> The second one separates the platform specific code in separate module and<br>
> won't affect ims_register_pcscf. However I need data from ims_usrloc_pcscf,<br>
> which is not accessible from the configuration. Also, writing separate<br>
> module for a limited IPSEC handling seems like a overkill for me.<br>
><br>
> What's your opinion?<br>
><br>
> Best regards,<br>
> Tsvetomir<br>
><br>
</div></div>> ______________________________<wbr>_________________<br>
> Kamailio (SER) - Development Mailing List<br>
> <a href="mailto:sr-dev@lists.kamailio.org">sr-dev@lists.kamailio.org</a><br>
> <a href="https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev" rel="noreferrer" target="_blank">https://lists.kamailio.org/<wbr>cgi-bin/mailman/listinfo/sr-<wbr>dev</a><br>
><br>
<br>
<br>
<br>
--<br>
Carsten Bock<br>
CEO (Geschäftsführer)<br>
<br>
ng-voice GmbH<br>
Millerntorplatz 1<br>
20359 Hamburg / Germany<br>
<br>
<a href="http://www.ng-voice.com" rel="noreferrer" target="_blank">http://www.ng-voice.com</a><br>
mailto:<a href="mailto:carsten@ng-voice.com">carsten@ng-voice.com</a><br>
<br>
Office <a href="tel:%2B49%2040%205247593-40" value="+4940524759340">+49 40 5247593-40</a><br>
Fax <a href="tel:%2B49%2040%205247593-99" value="+4940524759399">+49 40 5247593-99</a><br>
<br>
Sitz der Gesellschaft: Hamburg<br>
Registergericht: Amtsgericht Hamburg, HRB 120189<br>
Geschäftsführer: Carsten Bock<br>
Ust-ID: DE279344284<br>
<br>
Hier finden Sie unsere handelsrechtlichen Pflichtangaben:<br>
<a href="http://www.ng-voice.com/imprint/" rel="noreferrer" target="_blank">http://www.ng-voice.com/<wbr>imprint/</a><br>
<br>
______________________________<wbr>_________________<br>
Kamailio (SER) - Development Mailing List<br>
<a href="mailto:sr-dev@lists.kamailio.org">sr-dev@lists.kamailio.org</a><br>
<a href="https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev" rel="noreferrer" target="_blank">https://lists.kamailio.org/<wbr>cgi-bin/mailman/listinfo/sr-<wbr>dev</a><br>
</blockquote></div><br></div>