password-store

commit f97cf971d813bda250edaa3757c24ef1dc986309
parent 4a6fa5cbdf093d7873524fe566f58bca3761592e
Author: Allan Odgaard <github@simplit.com>
Date:   Thu, 14 Jun 2018 15:36:09 +0200

Close stdout for background task that restores clipboard

While we do not expect any output on stdout from the background task,
keeping the file handle open means that anyone calling `pass` and
waiting for stdout to be closed, will have to wait (by default) for 45
seconds.

Diffstat:
M
src/password-store.sh
 | 
2
+-
M
src/platform/cygwin.sh
 | 
2
+-
M
src/platform/darwin.sh
 | 
2
+-

3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/password-store.sh b/src/password-store.sh
@@ -173,7 +173,7 @@ clip() {
 		qdbus org.kde.klipper /klipper org.kde.klipper.klipper.clearClipboardHistory &>/dev/null
 
 		echo "$before" | base64 -d | xclip -selection "$X_SELECTION"
-	) 2>/dev/null & disown
+	) >/dev/null 2>&1 & disown
 	echo "Copied $2 to clipboard. Will clear in $CLIP_TIME seconds."
 }
 
diff --git a/src/platform/cygwin.sh b/src/platform/cygwin.sh
@@ -11,7 +11,7 @@ clip() {
 		local now="$(base64 < /dev/clipboard)"
 		[[ $now != $(echo -n "$1" | base64) ]] && before="$now"
 		echo "$before" | base64 -d > /dev/clipboard
-	) 2>/dev/null & disown
+	) >/dev/null 2>&1 & disown
 	echo "Copied $2 to clipboard. Will clear in $CLIP_TIME seconds."
 }
 
diff --git a/src/platform/darwin.sh b/src/platform/darwin.sh
@@ -11,7 +11,7 @@ clip() {
 		local now="$(pbpaste | openssl base64)"
 		[[ $now != $(echo -n "$1" | openssl base64) ]] && before="$now"
 		echo "$before" | openssl base64 -d | pbcopy
-	) 2>/dev/null & disown
+	) >/dev/null 2>&1 & disown
 	echo "Copied $2 to clipboard. Will clear in $CLIP_TIME seconds."
 }