Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • G gnome-session
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 52
    • Issues 52
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 10
    • Merge requests 10
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • GNOMEGNOME
  • gnome-session
  • Merge requests
  • !21

util: Blacklist some session-specific variables

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Iain Lane requested to merge wip/blacklist-session-specific-variables into master Aug 15, 2019
  • Overview 9
  • Commits 1
  • Pipelines 0
  • Changes 1

Things like XDG_SESSION_ID should not be uploaded to the environment. For example this is broken currently:

  1. SSH to your machine
  2. Log in to GNOME Shell
  3. Log out
  4. Log in again
  5. Lock the screen
  6. Try to unlock

You can't, and this is because the XDG_SESSION_ID from the first session (step 2) has leaked through to the second one (step 4), and so GNOME Shell is listening to the logind UnlockSession signal for the wrong session. The SSH session established in step 1 serves to keep the systemd --user instance alive, so that the state is not torn down between logins.

This is just one example of something that is broken currently, but it's likely that other things will be too in more or less subtle ways than this one. The idea is that we should stop setting things in the environment that are specific to one session and this is inspired by an example from D-Bus upstream.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: wip/blacklist-session-specific-variables