rpms/gnome-kiosk
7
0
Fork 0
Code Issues Pull Requests Releases Activity

Fix session registration

Browse Source 
Resolves: https://issues.redhat.com/browse/RHEL-153248
This commit is contained in:
Olivier Fourdan 2026-03-05 14:08:49 +01:00
parent ca78a8830e
commit 5cd58fc55e
2 changed files with 71 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

68
0001-sessions-Do-not-pretend-we-register-the-session.patch Normal file
View File

@ -0,0 +1,68 @@
From 02b9869f5040922768a9220f4ec2351769c4db59 Mon Sep 17 00:00:00 2001
From: Olivier Fourdan <ofourdan@redhat.com>
Date: Mon, 2 Feb 2026 11:27:12 +0100
Subject: [PATCH] sessions: Do not pretend we register the session
When it comes to the session, we can register the session ourselves or
leave that to GDM to sort out.
This is determined by the keyword "X-GDM-SessionRegister" in the session
file.
If that value is true, the compositor will register with the session
itself.
Unfortunately, GNOME Kiosk session have that wrong, the value is set to
true, yet GNOME Kiosk will not register with session itself unlike GNOME
Shell.
As a result, GDM will not terminate/restart the greeter, and after the
session terminates, it will reuse the old greeter. That prevents further
logins from GDM due to other issues in GDM.
To avoid that problem with GNOME Kiosk, we can either implement the
session registration in GNOME Kiosk, or simply change the value of the
key "X-GDM-SessionRegister" to false in the session file.
The latter being the simplest approach, this is the fix implemented
here.
Thanks to Joan Torres <joantolo@redhat.com> for helping me figuring out
the issue!
Closes: https://gitlab.gnome.org/GNOME/gnome-kiosk/-/issues/49
See-also: https://gitlab.gnome.org/GNOME/gdm/-/merge_requests/344
See-also: https://gitlab.gnome.org/GNOME/gdm/-/merge_requests/345
See-also: https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/3885
Part-of: <https://gitlab.gnome.org/GNOME/gnome-kiosk/-/merge_requests/119>
(cherry picked from commit ca15707997c2738e6d52a69c24f21d13d629ebed)
---
.../wayland-sessions/gnome-kiosk-script-wayland.desktop.in | 2 +-
.../org.gnome.Kiosk.SearchApp.Session.desktop.in | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/kiosk-script/wayland-sessions/gnome-kiosk-script-wayland.desktop.in b/kiosk-script/wayland-sessions/gnome-kiosk-script-wayland.desktop.in
index d170c72..b7f4455 100644
--- a/kiosk-script/wayland-sessions/gnome-kiosk-script-wayland.desktop.in
+++ b/kiosk-script/wayland-sessions/gnome-kiosk-script-wayland.desktop.in
@@ -5,5 +5,5 @@ Exec=gnome-session --session gnome-kiosk-script
TryExec=gnome-session
Type=Application
DesktopNames=GNOME-Kiosk;GNOME;
-X-GDM-SessionRegisters=true
+X-GDM-SessionRegisters=false
X-GDM-CanRunHeadless=true
diff --git a/search-app/wayland-sessions/org.gnome.Kiosk.SearchApp.Session.desktop.in b/search-app/wayland-sessions/org.gnome.Kiosk.SearchApp.Session.desktop.in
index 46ce73f..ac0a46a 100644
--- a/search-app/wayland-sessions/org.gnome.Kiosk.SearchApp.Session.desktop.in
+++ b/search-app/wayland-sessions/org.gnome.Kiosk.SearchApp.Session.desktop.in
@@ -5,5 +5,5 @@ Exec=gnome-session --session org.gnome.Kiosk.SearchApp
TryExec=gnome-session
Type=Application
DesktopNames=GNOME-Kiosk;GNOME;
-X-GDM-SessionRegisters=true
+X-GDM-SessionRegisters=false
X-GDM-CanRunHeadless=true
--
2.53.0

3
gnome-kiosk.spec
View File

@ -66,6 +66,9 @@ Patch: 0003-compositor-Disable-animations-w-out-HW-acceleration.patch
# https://issues.redhat.com/browse/RHEL-153245
Patch: 0001-systemd-sessions-Start-accessibility-settings.patch
# https://issues.redhat.com/browse/RHEL-153248
Patch: 0001-sessions-Do-not-pretend-we-register-the-session.patch
%description
GNOME Kiosk provides a desktop enviroment suitable for fixed purpose, or
single application deployments like wall displays and point-of-sale systems.