bug#74105: 30.0.92; Some issues in TRAMP NEWS entries

[Sean Whitton <spwhitton@spwhitton.name>]

X-debbugs-cc: michael.albinus@gmx.de

Hello,

In NEWS we have

    +++
    *** Direct asynchronous processes are indicated by a connection-local variable.
    If direct asynchronous processes shall be used, set the connection-local
    variable 'tramp-direct-async-process' to a non-nil value.  This has been
    changed, in previous Emacs versions this was indicated by the now
    deprecated connection property "direct-async-process".  See the Tramp
    manual "(tramp) Improving performance of asynchronous remote processes".

In my init.el I have this:

    (with-eval-after-load 'tramp
      (add-to-list 'tramp-connection-properties
                   '("/ssh:" "direct-async-process" t))
      (add-to-list 'tramp-connection-properties '(nil "session-timeout" nil)))

I have some documentation issues:

(1) The NEWS entry, together with the Info manual, were not sufficient
    for me to figure how to migrate my config.  For example, is the
    whole of tramp-connection-properties being replaced?  Or do I only
    need to migrate the direct-async-process part?

    I think additional documentation, even an explicit migration guide,
    would be desirable.

(2) Turning on direct async processes for all SSH connections seems like
    a common configuration.  How about adding that to the manual, in
    addition to the example for enabling them for one host?

(3) Has there been a change to how password caching works?

    I tried deleting the session-timeout configuration.  I made a
    /sudo:: connection.  After some time passed, I saw the familiar
    "Tramp connection timed out" message.  But it reconnected
    immediately when I tried to save a buffer I had open.  Does it now
    cache the sudo password?  In the past, you had to type it again,
    after the timeout.

    If there is a change here, it's security-relevant, so it should
    probably be in NEWS.

Thanks.

-- 
Sean Whitton