DelPort doesnt check if there is only 1 active port remaining

[DoctorD90]

Via IRC messages is possible to completely remove all listening ports.

In the code, here, should be there an other "IF", checking if the number of listeners is = 1; in that case it should give back error, telling to add first another port before to remove the last available port.

znc/src/ClientCommand.cpp

Lines 1718 to 1723 in 9cd3605

	if (pListener) {
	CZNC::Get().DelListener(pListener);
	PutStatus(t_s("Deleted Port"));
	} else {
	PutStatus(t_s("Unable to find a matching port"));
	}

[csprr]

The if should be to check if it's the current active port, because that one should never be able to be deleted (which is already the case in webadmin iirc)

[DarthGandalf]

This is mostly by design. It should be possible to e.g. switch SSL on on the port without adding a new port with different number: delport, then addport. Of course, there's a potential to disconnect after delport, which would make ZNC not accessible anymore.

Webadmin does have this check, because all HTTP connections are independent (we always use Connection: close header, and even without it, we can't rely on browser to not close the connection between requests), so deleting the port from there would result in a weird error instead of the "success page"