Bug 14053 - Please create pam_console compatible tag files
Summary: Please create pam_console compatible tag files
Status: NEW
Alias: None
Product: ConsoleKit
Classification: Unclassified
Component: Daemon (show other bugs)
Version: unspecified
Hardware: All All
: medium enhancement
Assignee: william.jon.mccann
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-01-13 04:50 UTC by Martin Pitt
Modified: 2008-07-16 15:24 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
implement /var/run/console/<username> tags (4.00 KB, patch)
2008-01-13 04:50 UTC, Martin Pitt
Details | Splinter Review
implement /var/run/console/<username> tags (4.83 KB, patch)
2008-01-30 06:47 UTC, Martin Pitt
Details | Splinter Review
run-session.d script (446 bytes, text/plain)
2008-07-16 15:24 UTC, Martin Pitt
Details

Description Martin Pitt 2008-01-13 04:50:12 UTC
Created attachment 13690 [details] [review]
implement /var/run/console/<username> tags

There are currently some applications which use the dbus policy "at_console". In order to drop libpam-console, libpam-foreground & friends and only use ConsoleKit (which already has this information anyway), it would be nice if CK would write pam_console like tag files (/var/run/console/<username>).

I attached the Ubuntu patch for doing this.

Thank you for considering,

Martin
Comment 1 Martin Pitt 2008-01-13 04:53:28 UTC
Oh, I forgot to mention: It would be bad style if dbus itself would issue dbus calls to CK to check if a caller is at a local console.
Comment 2 william.jon.mccann 2008-01-28 19:32:09 UTC
I think the way we want to do this is to use scripts in run-session.d for this.
Comment 3 Martin Pitt 2008-01-29 01:53:27 UTC
Oh, I was not aware that CK already provides a hooks.d directory (the Debian package doesn't ship one either). I'm not much fussed about my patch, you are welcome to throw it away, but we need it working one way or the other until all projects are converted to not use dbus at_console policy any more.

Thank you!
Comment 4 Martin Pitt 2008-01-30 06:47:51 UTC
Created attachment 14034 [details] [review]
implement /var/run/console/<username> tags

This updated patch creates the PAM console stamps for local CK sessions only.
Comment 5 Martin Pitt 2008-07-16 15:24:53 UTC
Created attachment 17717 [details]
run-session.d script

I threw away the patch in Ubuntu now and replaced it with a run-session.d/ script. Thanks for the hint!


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.