| Subject: | Documentation is misleading in NET::MitM 0.02 |
| Date: | Tue, 29 Jan 2019 12:57:01 +0100 |
| To: | bug-NET-MitM [...] rt.cpan.org |
| From: | Csaba Raduly <rcsaba [...] gmail.com> |
Hi,
The documentation for the "new" method says
new( remote_ip_address, local_port_num, remote_port_num )
However, this is contradicted by the order in which the parameters are
described:
Parameters
* remote_ip_address - the remote hostname/IP address of the server
* remote_port_num - the remote port number of the server
* local_port_num - the port number to listen on
and also the example
NET::MitM->new("www.cpan.org", 80, 10080);
where it's clear that 10080 is the local port the proxy listens on.
It's also inconsistent with the implementation of new()
I'm using Ubuntu 18.04.1 LTS
$ perl -v
This is perl 5, version 26, subversion 1 (v5.26.1) built for
x86_64-linux-gnu-thread-multi
(with 67 registered patches, see perl -V for more detail)
Csaba
--
You can get very substantial performance improvements
by not doing the right thing. - Scott Meyers, An Effective C++11/14 Sampler
So if you're looking for a completely portable, 100% standards-conformat way
to get the wrong information: this is what you want. - Scott Meyers (C++TDaWYK)