After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 791200 - nmcli previous password is shown if you press up arrow key on password reprompt
nmcli previous password is shown if you press up arrow key on password reprompt
Status: RESOLVED FIXED
Product: NetworkManager
Classification: Platform
Component: nmcli
1.10.x
Other Linux
: Normal normal
: ---
Assigned To: Lubomir Rintel
NetworkManager maintainer(s)
https://asciinema.org/a/150932
Depends on:
Blocks: nm-review
 
 
Reported: 2017-12-04 11:25 UTC by suizman
Modified: 2017-12-18 13:52 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
[PATCH 1/3] clients/cli: don't store a password in history (1.57 KB, text/plain)
2017-12-15 16:17 UTC, Lubomir Rintel
Details
[PATCH 2/3] libnm-core: move detection of UTF-8 capable terminals to clients/ (6.93 KB, text/plain)
2017-12-15 16:18 UTC, Lubomir Rintel
Details
[PATCH 3/3] clients/cli: use a nicer password prompt (4.08 KB, text/plain)
2017-12-15 16:18 UTC, Lubomir Rintel
Details
[PATCH 3/3] clients/cli: use a nicer password prompt (4.33 KB, text/plain)
2017-12-15 16:21 UTC, Lubomir Rintel
Details
[PATCH 3/3] clients/cli: use a nicer password prompt (4.31 KB, text/plain)
2017-12-15 16:25 UTC, Lubomir Rintel
Details

Description suizman 2017-12-04 11:25:59 UTC
When nmcli reprompts to input your password, if you press up arrow key the previous password is shown.


How to reproduce
https://asciinema.org/a/150932
Comment 1 Lubomir Rintel 2017-12-15 16:17:48 UTC
Created attachment 365593 [details]
[PATCH 1/3] clients/cli: don't store a password in history
Comment 2 Lubomir Rintel 2017-12-15 16:18:13 UTC
Created attachment 365594 [details]
[PATCH 2/3] libnm-core: move detection of UTF-8 capable terminals to clients/
Comment 3 Lubomir Rintel 2017-12-15 16:18:45 UTC
Created attachment 365595 [details]
[PATCH 3/3] clients/cli: use a nicer password prompt
Comment 4 Lubomir Rintel 2017-12-15 16:19:31 UTC
Thanks for the report.

Fix ready for review, with some extra eye-candy.
Comment 5 Lubomir Rintel 2017-12-15 16:21:07 UTC
Created attachment 365596 [details]
[PATCH 3/3] clients/cli: use a nicer password prompt
Comment 6 Lubomir Rintel 2017-12-15 16:25:51 UTC
Created attachment 365597 [details]
[PATCH 3/3] clients/cli: use a nicer password prompt
Comment 7 suizman 2017-12-15 16:28:32 UTC
(In reply to Lubomir Rintel from comment #4)
> Thanks for the report.
> 
> Fix ready for review, with some extra eye-candy.

You're welcome. Thanks for the fix!
Comment 8 Beniamino Galvani 2017-12-18 09:44:37 UTC
> libnm-core: move detection of UTF-8 capable terminals to clients/

nmt-connect-connection-list.c needs to include nm-client-utils.h


> clients/cli: use a nicer password prompt

+static void
+nmc_secret_redisplay ()

Needs a 'void' argument, otherwise:

clients/cli/common.c:883:1: error: function declaration isn’t a prototype [-Werror=strict-prototypes]
Comment 9 Thomas Haller 2017-12-18 12:46:36 UTC
This just avoids awkwars surprises,
                 ^^^^^^^


+ what bg said.

rest lgtm
Comment 10 Lubomir Rintel 2017-12-18 13:52:20 UTC
This problem has been fixed in the unstable development version. The fix will be available in the next major software release. You may need to upgrade your Linux distribution to obtain that newer version.