<div dir="ltr">Kevin,<div><br></div><div>Yes, this inherited the base code from nx-firewall.</div><div>I'll see how easy is to implement this on the current codebase without missing deadline for plasma 21.</div><div><br></div><div>Tomaz</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jan 22, 2021 at 11:54 PM Kevin Kofler <<a href="mailto:kevin.kofler@chello.at">kevin.kofler@chello.at</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Tomaz Canabrava wrote:<br>
> Please review the code and test.<br>
<br>
Unfortunately, this appears to have inherited the usability issues from<br>
nx-firewall:<br>
<a href="https://invent.kde.org/network/plasma-firewall/-/issues/14" rel="noreferrer" target="_blank">https://invent.kde.org/network/plasma-firewall/-/issues/14</a><br>
<a href="https://invent.kde.org/network/plasma-firewall/-/issues/15" rel="noreferrer" target="_blank">https://invent.kde.org/network/plasma-firewall/-/issues/15</a><br>
and a new one comes up with the added support for firewalld:<br>
<a href="https://invent.kde.org/network/plasma-firewall/-/issues/16" rel="noreferrer" target="_blank">https://invent.kde.org/network/plasma-firewall/-/issues/16</a><br>
<br>
IMHO, support for application profiles (#14) is essential for a usable <br>
firewall UI. (IMHO, it is the absolute minimum. For firewalld, you also need <br>
to handle the concept of zones sooner or later, see #16.)<br>
<br>
A simpler way to create simple rules (only setting the incoming port, <br>
without needing the advanced settings) (#15) would also be helpful, though <br>
you may be able to get away without it if you address #14.<br>
<br>
        Kevin Kofler<br>
</blockquote></div>