Closed karldreher closed 4 years ago
fixed in #21 and https://github.com/karldreher/KnowYourPorts/pull/21/commits/be620dc81e7d9cc2630afa46b762c253b376034f.
The way this was fixed was by deprecation of this functionality. As discussed in the commit log, I believe there are better ways to do this, and would like to focus this project on the web functionality.
I can't decide if I actually hate this or not, but open to some feedback on the subject.
The primary value of the console app is for scripting purposes, in which case the script developer should be able to work around it. However, the inclusion of a command line switch to choose output could be helpful as well. Secondarily, this is a good "reference" app which can be used quickly in a terminal, so the default setting should be an easy to read format from which an administrator can quickly derive information.
Ideally (this was the way it functioned before), output is the port number, a tab, and the port description. I broke this in order to get the web app working, and was OK with it at the time.