LightDMUser :: get_user_image_cb() :: Only pass value to themes if file exists and is accessible about web-greeter HOT 5 CLOSED

http://people.ubuntu.com/~robert-ancell/lightdm/reference/lightdm-gobject-1-User-Accounts.html#lightdm-user-get-image

We're basically just providing the result of the lightdm callback, so it should already be that way. Are you seeing something different?

EDIT: Ah, I see what you're wanting; the greeter runs as the lightdm user; so just because we've had the uri handed to us, doesn't mean it's accessible. OK, we just need an access check. I'll add it.

from web-greeter.

This might actually be a bug in lightdm, I am not sure yet. While testing the theme yesterday I noticed that lightdm was providing a user image filepath that didn't actually exist: /home/antergos/.face I know that's the default place it looks for the image but there was no image for my test (which was a fresh installation in a VM). If I see the issue again I will report it upstream, but I figure it doesnt hurt to have a check on our end in the meantime.

from web-greeter.

My fix works for me, but now you'll have to fix the antergos theme; at one point, you do a .length, and it complains when it's NULL.

from web-greeter.

Also, check out my newer commits. The text we receive out of lightdm via the show_message() callback is just plain text; it's got newlines, &'s potentially <'s and >'s depending on what messages any particular pam module author decides to spit at us. To make the theme author's work a bit easier, I've added a greeterutil.txt2html() function which will clean up passed text. Saves the author having to manually make it html "clean" in javascript.

from web-greeter.

Fixed in 64f4839

from web-greeter.