I messed things up in 48de4c0d240579fe8cfd71c5dff177ba14f78d3e
("Remove the 'verbose' global variable"), where I failed to account
for the fact that `vpninfo->verbose` defaults to `PRG_TRACE`, and that
it needs to be set (via `openconnect_set_loglevel`) immediately after
option parsing is complete.
Signed-off-by: Daniel Lenski <email address hidden>
When we use select(), an error event on a file descriptor causes it to
be reported as readable, and some of the handlers will only clean it up
if that happens. So set the 'readable' flag on EPOLLERR too.
This is part of #389.
Signed-off-by: David Woodhouse <email address hidden>
63e4b85...
by
Dimitri Papadopoulos <email address hidden>
However, we forget to make stoken.c set the `auth_id` for the forms that it
creates for RSA SecurID token decryption and PIN entry. Let's name these:
- `_rsa_unlock`, for token decryption.
- `_rsa_pin`, for PIN entry. Also, rename the numeric PIN field to `pin`
rather than `password`; there can't be any existing users relying on
`--form-entry` to set its value, because that wouldn't work without the
`auth_id`.
Fixes #388.
Signed-off-by: Daniel Lenski <email address hidden>