<div dir="ltr"><div>Package: strongswan</div><div>Version: 5.5.1-2</div><div>Severity: normal</div><div><br></div><div>Hi,</div><div>this is the part of the feed back of "Ubuntu changes" that is probably more discussion worthy.</div><div>And I'm not even speaking of renamed service names yet :-)</div><div><br></div><div>Our Ubuntu Delta for quite a while dropped some outdated code.</div><div>I didn't come up with the wording - it was called outdated before I even touched Strongswan - so no offense intended.</div><div><br></div><div>Since I was not 100% sure if I consider all special cases that might apply to I'm only starting a discussion about it and we can see if you agree (instead of submitting patches).</div><div><br></div>In d/strongswan-starter.postinst there seems to be code that is no more needed:<br>  - section about runlevel changes<br>  - entire section on opportunistic encryption - this was never used<div><br></div><div><div>Also the debconf for the inc file is IMHO not working.</div><div><div>cat /var/lib/strongswan/ipsec.conf.inc</div><div>      #include /etc/ipsec.d/examples/oe.conf</div></div><div>That file is not existing (and not created) on my Debian-sid system.</div><div><br></div><div>I tried to get as much as possible from its debconf but it does only add the disabled OE.</div><div><div>$ dpkg-reconfigure strongswan-starter -p low<br></div><div>  Disabling opportunistic encryption (OE) in config file ... already disabled</div></div><div><br></div><div><br></div><div>All that together would make up some nice cleanup like the attached patch I have as a suggestion.</div><div>Looking forward to a discussion on all of that.</div><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><span style="color:rgb(136,136,136);font-size:12.8px">Christian Ehrhardt</span><div style="color:rgb(136,136,136);font-size:12.8px">Software Engineer, Ubuntu Server</div><div style="color:rgb(136,136,136);font-size:12.8px">Canonical Ltd</div></div></div></div></div>
</div></div>