Rationalise and document log options somewhat.

TOOLTYPE_NONNETWORK (i.e. pterm) already has "-log" (as does Unix
PuTTY), so there's no sense suppressing the synonym "-sessionlog".

Undocumented lacunae that remain:

plink accepts -sessionlog, but does nothing with it. Arguably it should.

puttytel accepts -sshlog/-sshrawlog (and happily logs e.g. Telnet
negotiation, as does PuTTY proper).

diff --git a/cmdline.c b/cmdline.c
index fbf1e5c9e69710fce26e19320f076a314d921e27..92c87b3b797b61da4cd5a171699983377d10cc88 100644 (file)
--- a/cmdline.c
+++ b/cmdline.c
@@ -575,8 +575,19 @@ int cmdline_process_param(const char *p, char *value,
        }
     }
 
-    if (!strcmp(p, "-sessionlog") ||
-        !strcmp(p, "-sshlog") ||
+    if (!strcmp(p, "-sessionlog")) {
+       Filename *fn;
+       RETURN(2);
+       UNAVAILABLE_IN(TOOLTYPE_FILETRANSFER);
+       /* but available even in TOOLTYPE_NONNETWORK, cf pterm "-log" */
+       SAVEABLE(0);
+       fn = filename_from_str(value);
+       conf_set_filename(conf, CONF_logfilename, fn);
+       conf_set_int(conf, CONF_logtype, LGTYP_DEBUG);
+        filename_free(fn);
+    }
+
+    if (!strcmp(p, "-sshlog") ||
         !strcmp(p, "-sshrawlog")) {
        Filename *fn;
        RETURN(2);
@@ -585,7 +596,6 @@ int cmdline_process_param(const char *p, char *value,
        fn = filename_from_str(value);
        conf_set_filename(conf, CONF_logfilename, fn);
        conf_set_int(conf, CONF_logtype,
-                     !strcmp(p, "-sessionlog") ? LGTYP_DEBUG :
                      !strcmp(p, "-sshlog") ? LGTYP_PACKETS :
                      /* !strcmp(p, "-sshrawlog") ? */ LGTYP_SSHRAW);
         filename_free(fn);

diff --git a/doc/man-pl.but b/doc/man-pl.but
index 6a358ceaac314fb8f9b693e261ccb878a725b1f1..b597b83f923cb8370a11c7693b7ef6c6f1d993ef 100644 (file)
--- a/doc/man-pl.but
+++ b/doc/man-pl.but
@@ -184,6 +184,20 @@ DSR/DTR.
 
 }
 
+\dt \cw{\-sshlog} \e{logfile}
+
+\dt \cw{\-sshrawlog} \e{logfile}
+
+\dd For SSH connections, these options make \cw{plink} log protocol
+details to a file. (Some of these may be sensitive, although by default
+an effort is made to suppress obvious passwords.)
+
+\lcont{
+\cw{\-sshlog} logs decoded SSH packets and other events (those that
+\cw{\-v} would print). \cw{\-sshrawlog} additionally logs the raw
+encrypted packet data.
+}
+
 \S{plink-manpage-more-information} MORE INFORMATION
 
 For more information on plink, it's probably best to go and look at

diff --git a/doc/man-pscp.but b/doc/man-pscp.but
index 5cd5ac8cd33c50a4b0e1a6a12cc669f5d292417a..1b95d4d243f9c3af6015d7196e80dc24e5153c32 100644 (file)
--- a/doc/man-pscp.but
+++ b/doc/man-pscp.but
@@ -115,6 +115,20 @@ written. }
 
 \dd Force use of SFTP protocol.
 
+\dt \cw{\-sshlog} \e{logfile}
+
+\dt \cw{\-sshrawlog} \e{logfile}
+
+\dd These options make \cw{pscp} log protocol details to a file.
+(Some of these may be sensitive, although by default an effort is made
+to suppress obvious passwords.)
+
+\lcont{
+\cw{\-sshlog} logs decoded SSH packets and other events (those that
+\cw{\-v} would print). \cw{\-sshrawlog} additionally logs the raw
+encrypted packet data.
+}
+
 \S{pscp-manpage-more-information} MORE INFORMATION
 
 For more information on \cw{pscp} it's probably best to go and look at

diff --git a/doc/man-psft.but b/doc/man-psft.but
index f24d7206c4a10e09f33ee4abf83bc29aca95a05c..0194779a7c26a269fee37f34ddcf78d679d571e6 100644 (file)
--- a/doc/man-psft.but
+++ b/doc/man-psft.but
@@ -95,6 +95,20 @@ accepted (unless a saved session also overrides host keys, in which
 case those will be added to), and the host key cache will not be
 written. }
 
+\dt \cw{\-sshlog} \e{logfile}
+
+\dt \cw{\-sshrawlog} \e{logfile}
+
+\dd These options make \cw{psftp} log protocol details to a file.
+(Some of these may be sensitive, although by default an effort is made
+to suppress obvious passwords.)
+
+\lcont{
+\cw{\-sshlog} logs decoded SSH packets and other events (those that
+\cw{\-v} would print). \cw{\-sshrawlog} additionally logs the raw
+encrypted packet data.
+}
+
 \S{psftp-manpage-commands} COMMANDS
 
 For a list of commands available inside \cw{psftp}, type \cw{help}

diff --git a/doc/man-ptel.but b/doc/man-ptel.but
index c0ee9c64eb0cdbbf74faa65c22fc2c6947ad8acd..07a756b112a54eff32701b40807a595652817955 100644 (file)
--- a/doc/man-ptel.but
+++ b/doc/man-ptel.but
@@ -109,7 +109,7 @@ changed under control of the server.)
 to specify it explicitly if you have changed the default using the
 \cw{ScrollBar} resource.
 
-\dt \cw{\-log} \e{filename}
+\dt \cw{\-log} \e{logfile}, \cw{\-sessionlog} \e{logfile}
 
 \dd This option makes \cw{puttytel} log all the terminal output to a file
 as well as displaying it in the terminal.

diff --git a/doc/man-pter.but b/doc/man-pter.but
index a357e3e397ae845d3b3a48ce229e0ba77d1aa1d9..26d5842d06f42065aac1c14096a33a8ddf0093ce 100644 (file)
--- a/doc/man-pter.but
+++ b/doc/man-pter.but
@@ -153,7 +153,7 @@ default using the \cw{LoginShell} resource.
 to specify it explicitly if you have changed the default using the
 \cw{ScrollBar} resource.
 
-\dt \cw{\-log} \e{filename}
+\dt \cw{\-log} \e{logfile}, \cw{\-sessionlog} \e{logfile}
 
 \dd This option makes \cw{pterm} log all the terminal output to a file
 as well as displaying it in the terminal.

diff --git a/doc/man-putt.but b/doc/man-putt.but
index 76694def8ee112e249b10d101cf6b5a45176b771..67bcba83c0fa9338c42b8ce5edc4d880323a2f7e 100644 (file)
--- a/doc/man-putt.but
+++ b/doc/man-putt.but
@@ -108,11 +108,24 @@ changed under control of the server.)
 to specify it explicitly if you have changed the default using the
 \cw{ScrollBar} resource.
 
-\dt \cw{\-log} \e{filename}
+\dt \cw{\-log} \e{logfile}, \cw{\-sessionlog} \e{logfile}
 
 \dd This option makes \cw{putty} log all the terminal output to a file
 as well as displaying it in the terminal.
 
+\dt \cw{\-sshlog} \e{logfile}
+
+\dt \cw{\-sshrawlog} \e{logfile}
+
+\dd For SSH connections, these options make \cw{putty} log protocol
+details to a file. (Some of these may be sensitive, although by default
+an effort is made to suppress obvious passwords.)
+
+\lcont{
+\cw{\-sshlog} logs decoded SSH packets and other events (those that
+\cw{\-v} would print). \cw{\-sshrawlog} additionally logs the raw
+encrypted packet data.
+}
 
 \dt \cw{\-cs} \e{charset}
 

diff --git a/pscp.c b/pscp.c
index 0b37def18797b250383c890f14ff54237f27238a..a4e55fe09b457be378152051125797071afe9b60 100644 (file)
--- a/pscp.c
+++ b/pscp.c
@@ -2239,6 +2239,9 @@ static void usage(void)
     printf("  -unsafe   allow server-side wildcards (DANGEROUS)\n");
     printf("  -sftp     force use of SFTP protocol\n");
     printf("  -scp      force use of SCP protocol\n");
+    printf("  -sshlog file\n");
+    printf("  -sshrawlog file\n");
+    printf("            log protocol details to a file\n");
 #if 0
     /*
      * -gui is an internal option, used by GUI front ends to get

diff --git a/psftp.c b/psftp.c
index 93a96a4e152a63cc682042474a9a1c0c8d2573fc..92b57a2f363793b94b95aa9aa2128505dd13b6a0 100644 (file)
--- a/psftp.c
+++ b/psftp.c
@@ -2647,6 +2647,9 @@ static void usage(void)
     printf("  -hostkey aa:bb:cc:...\n");
     printf("            manually specify a host key (may be repeated)\n");
     printf("  -batch    disable all interactive prompts\n");
+    printf("  -sshlog file\n");
+    printf("  -sshrawlog file\n");
+    printf("            log protocol details to a file\n");
     cleanup_exit(1);
 }
 

diff --git a/unix/gtkwin.c b/unix/gtkwin.c
index 872e6bb94f5f074e3f744028efe8f275bc8f2daf..b1c86dd896aff5e5423f4a4146a402bcfacfa725 100644 (file)
--- a/unix/gtkwin.c
+++ b/unix/gtkwin.c
@@ -3749,7 +3749,7 @@ static void help(FILE *fp) {
 "  -ut, +ut                  Do(default) or do not update utmp\n"
 "  -ls, +ls                  Do(default) or do not make shell a login shell\n"
 "  -sb, +sb                  Do(default) or do not display a scrollbar\n"
-"  -log PATH                 Log all output to a file\n"
+"  -log PATH, -sessionlog PATH  Log all output to a file\n"
 "  -nethack                  Map numeric keypad to hjklyubn direction keys\n"
 "  -xrm RESOURCE-STRING      Set an X resource\n"
 "  -e COMMAND [ARGS...]      Execute command (consumes all remaining args)\n"

diff --git a/unix/uxplink.c b/unix/uxplink.c
index b8c554894002033dcb0e5583fd0285b9223bf11c..3d3a645fbbebae48dfdb339362f185a1314a7f84 100644 (file)
--- a/unix/uxplink.c
+++ b/unix/uxplink.c
@@ -581,6 +581,9 @@ static void usage(void)
     printf("  -N        don't start a shell/command (SSH-2 only)\n");
     printf("  -nc host:port\n");
     printf("            open tunnel in place of session (SSH-2 only)\n");
+    printf("  -sshlog file\n");
+    printf("  -sshrawlog file\n");
+    printf("            log protocol details to a file\n");
     printf("  -shareexists\n");
     printf("            test whether a connection-sharing upstream exists\n");
     exit(1);

diff --git a/windows/winplink.c b/windows/winplink.c
index 78311b59ed7fa5c424905f87dd1fbcbe0de6e053..ac4dab299fe19ffb0bb5de3f268da1b904feab7f 100644 (file)
--- a/windows/winplink.c
+++ b/windows/winplink.c
@@ -212,6 +212,9 @@ static void usage(void)
     printf("  -N        don't start a shell/command (SSH-2 only)\n");
     printf("  -nc host:port\n");
     printf("            open tunnel in place of session (SSH-2 only)\n");
+    printf("  -sshlog file\n");
+    printf("  -sshrawlog file\n");
+    printf("            log protocol details to a file\n");
     printf("  -shareexists\n");
     printf("            test whether a connection-sharing upstream exists\n");
     exit(1);