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 698200 - No longer displays realmd domain hint
No longer displays realmd domain hint
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: login-screen
3.8.x
Other Linux
: Normal normal
: ---
Assigned To: Ray Strode [halfline]
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2013-04-17 13:58 UTC by Stef Walter
Modified: 2013-05-03 19:57 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
gdm: Fix regression where domain login hint not shown (6.08 KB, patch)
2013-04-19 07:08 UTC, Stef Walter
committed Details | Review

Description Stef Walter 2013-04-17 13:58:47 UTC
The hint once joined to a domain is no longer being emitted in the GDM 'User name' screen.

I think that since this commit: 1ec0665b631965b8c73277374dddde8b1d8b6d1e

The InfoQuery signal is not being emitted (which causes the the code in util.js listening to 'info-query' not to be run).
Comment 1 Ray Strode [halfline] 2013-04-17 14:15:19 UTC
So commit 1ec0665b631965b8c73277374dddde8b1d8b6d1e is about porting the fallback greeter to libgdm. We don't use the fallback greeter anymore (and the fallback greeter never showed realmd hints).

The InfoQuery signal is emitted whenever a non-secret username is requested by PAM.  The canonical info-query is "Username: ".   This is probably caused by this commit:

https://git.gnome.org/browse/gnome-shell/commit/?id=1ae0fadbf483128e27565baba6e8684bc5defca2

We probably need to move the realmd hit from _onInfoQuery to _askForUsernameAndLogIn:
Comment 2 Ray Strode [halfline] 2013-04-17 14:16:07 UTC
above where i wrote "non-secret username" I meant to say "non-secret question"
Comment 3 Stef Walter 2013-04-19 07:08:29 UTC
Created attachment 241879 [details] [review]
gdm: Fix regression where domain login hint not shown

Also only keep around realmd while we're actually using it, allow it
to quit if no other clients are active.
Comment 4 Stef Walter 2013-04-19 07:09:55 UTC
Patch tested on gnome-3-8, applies to master.
Comment 5 Baptiste Mille-Mathias 2013-04-26 16:07:03 UTC
Stef or Ray,

Is this bug fixed ? does it require the backport of the patch on gnome-3-8 ?

Regards
Comment 6 Stef Walter 2013-04-26 16:33:45 UTC
I don't think this is committed yet. It needs review. Ideally I'd like it to get on both the gnome-3-8 branch and master.
Comment 7 Florian Müllner 2013-05-03 19:46:55 UTC
Review of attachment 241879 [details] [review]:

Looks good to me, though I haven't been able to actually test this.
Comment 8 Stef Walter 2013-05-03 19:57:29 UTC
Attachment 241879 [details] pushed as 47dee22 - gdm: Fix regression where domain login hint not shown

Thanks for the review. Pushed to gnome-3-8, and will push to master shortly.