[Bug 3328] Issue with ForwardAgent value specified as an environment variable

[bugzilla-daemon at mindrot.org]

https://bugzilla.mindrot.org/show_bug.cgi?id=3328

--- Comment #1 from Darren Tucker <dtucker at dtucker.net> ---
(In reply to Stephen Goetze from comment #0)
[...]
> I've attached a proposed patch, based on the OpenSSH 8.6p1 ssh.c
> source file.

Thanks, I've added it to the list to do for the next release.

> On a related note, I don't understand why the $ENV_VAR_NAME (without
> braces) syntax is supported for this option.  There is also support
> for supplying the environment variable name via the ${ENV_VAR_NAME}
> (with braces) syntax (see the code beginning at line 1415 in ssh.c).
> 
> 
> Is the non-brace syntax a legacy format that needs to be preserved?

Yes, the non-brace syntax was existed only for this option prior to the
more generic brace support being added for many options.  The old
syntax is maintained for backward compatibility for any configs using
it (we try not to break working configs without advance warning).

-- 
You are receiving this mail because:
You are watching someone on the CC list of the bug.
You are watching the assignee of the bug.