[gnome-shell] gdm: Always show fingerprint error messages
- From: Marge Bot <marge-bot src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-shell] gdm: Always show fingerprint error messages
- Date: Mon, 15 Feb 2021 17:05:27 +0000 (UTC)
commit 53db4b99b83dd8c885b966742370cc0348cfb6f3
Author: Marco Trevisan (TreviƱo) <mail 3v1n0 net>
Date: Tue Nov 24 19:19:22 2020 +0100
gdm: Always show fingerprint error messages
When the login/lock screen is shown the error messages for background
services are always ignored.
However, in case the service is the fingerprint authentication method
we still want to be able to show error messages to inform the user
about what failed, and eventually that the max retries (that may be
different from the login screen configuration) has been reached.
This handles partially the design issue [1] related to the login/lock
screen fingerprint authentication.
Eventually we want to use pam extensions to use clearer and parse-able
messages, however in the case of the fingerprint service we can be sure
that the fprint PAM module will only send errors on auth failures.
[1] https://gitlab.gnome.org/Teams/Design/os-mockups/-/issues/56
Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/1652>
js/gdm/util.js | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
---
diff --git a/js/gdm/util.js b/js/gdm/util.js
index c53fbd2176..969e126682 100644
--- a/js/gdm/util.js
+++ b/js/gdm/util.js
@@ -549,7 +549,9 @@ var ShellUserVerifier = class {
}
_onProblem(client, serviceName, problem) {
- if (!this.serviceIsForeground(serviceName))
+ const isFingerprint = this.serviceIsFingerprint(serviceName);
+
+ if (!this.serviceIsForeground(serviceName) && !isFingerprint)
return;
this._queueMessage(problem, MessageType.ERROR);
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]