threads / patch / 54596

patchgit-gui: ssh-askpass: add a checkbox to show the input text

Subject: [PATCH] git-gui: ssh-askpass: add a checkbox to show the input text

## tl;dr

2 messages between Nov 7, 2020 and Dec 1, 2020. Diffs are folded; open one to read it.

replies: 1people: 2as markdown or json

David Aguilar· Nov 7, 2020, 22:20 UTC · lore

Hide the input text by default since the field is commonly used for sensative informations such as passwords.

Add a "Show input" checkbox to conditionally show the input.
Helped-by: Miguel Boekhold <miguel.boekhold@osudio.com>
Signed-off-by: Efimov Vasily <laer.18@gmail.com>
Signed-off-by: David Aguilar <davvid@gmail.com>
---
 git-gui--askpass | 15 ++++++++++++++-
 1 file changed, 14 insertions(+), 1 deletion(-)
This patch gathers the submissions from the following sources:

https://github.com/patthoyts/git-gui/issues/22 https://github.com/prati0100/git-gui/pull/38 https://github.com/git-cola/git-cola/issues/1004 https://github.com/git-cola/git-cola/pull/1069

Show changes to git-gui--askpass +14 −1
diff --git a/git-gui--askpass b/git-gui--askpass
index 1c99ee8..71a536d 100755
--- a/git-gui--askpass
+++ b/git-gui--askpass
@@ -26,8 +26,21 @@ pack .m -side top -fill x -padx 20 -pady 20 -expand 1
 entry .e -textvariable answer -width 50
 pack .e -side top -fill x -padx 10 -pady 10
 
+proc on_show_input_changed {args} {
+	global show_input
+	if {$show_input} {
+		.e configure -show ""
+	} else {
+		.e configure -show "*"
+	}
+}
+trace add variable show_input write "on_show_input_changed"
+
+set show_input 0
+
 if {!$yesno} {
-	.e configure -show "*"
+	checkbutton .cb_show -text "Show input" -variable show_input
+	pack .cb_show -side top -anchor nw
 }
 
 frame .b
-- 
2.29.2.154.g7f7ebe054a
Pratyush Yadav· Dec 1, 2020, 19:41 UTC · re: David Aguilar · lore

Re: [PATCH] git-gui: ssh-askpass: add a checkbox to show the input text

On 07/11/20 02:20PM, David Aguilar wrote:
Show 8 quoted lines
> Hide the input text by default since the field is
> commonly used for sensative informations such as passwords.
> 
> Add a "Show input" checkbox to conditionally show the input.
> 
> Helped-by: Miguel Boekhold <miguel.boekhold@osudio.com>
> Signed-off-by: Efimov Vasily <laer.18@gmail.com>
> Signed-off-by: David Aguilar <davvid@gmail.com>
Merged to git-gui/master. Thanks.
-- 
Regards,
Pratyush Yadav

← back to recent threads