Skip to content

Conversation

@lcniel
Copy link

@lcniel lcniel commented Dec 4, 2025

See discussion: #3521

The motivation behind this PR is to allow a user (or an automation script) to maintain multiple distinct sessions based on the port configuration used when the session is started up. The reason for this is to allow xrdp to imitate the behaviour of multiple VNC servers started by users, while allowing a single instance of xrdp-sesman to track the sessions, and letting users (or middleware) choose which session to connect to based on the port specified on connection.

The specific use-case I have for this integrating xrdp with e.g., OpenOnDemand to allow users to start multiple remote desktop sessions without having to worry about breakage if two sessions by coincidence start on the same compute node.

In my testing so far, using the new P policy works just as well as the D policy - one has to ensure DBUS and XDG_RUNTIME_DIR don't get in the way of multiple xfce4 sessions, but the distinction of sessions by xrdp port works just fine.

This is just my suggestion for how to implement it, I expect that it will have to go through a few rounds of reviewing and editing.

@matt335672
Copy link
Member

Thanks for this @lcniel - looks good.

I'm a bit snowed under with stuff IRL at the moment, but I hope to look at this early next week.

If you rebase to the latest devel when we've had a review, it will pull in #3684 to fix the FreeBSD CI issue.

Remove trailing space
@lcniel
Copy link
Author

lcniel commented Dec 5, 2025

Thanks for this @lcniel - looks good.

I'm a bit snowed under with stuff IRL at the moment, but I hope to look at this early next week.

If you rebase to the latest devel when we've had a review, it will pull in #3684 to fix the FreeBSD CI issue.

Thanks, no rush at all. Hopefully I will be able to contribute a bit to the project moving forward since we're planning to use it quite a bit at my job.

@matt335672
Copy link
Member

On a personal note, I got into xrdp coding because I was the UNIX administrator for a network where it was the best solution for what we were doing. So I'm more that happy to help you with this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants