Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support MacOS. #2812

Open
wants to merge 1 commit into
base: devel
Choose a base branch
from
Open

Support MacOS. #2812

wants to merge 1 commit into from

Conversation

mstone2001
Copy link

This is 4 of 4 of my OS changes. As per comments I'm going to look at removing the daemondo program and see if it will work with manilla plist files.

MacOS:

Added plist files to run xrdp and xrdp-sesman. Note these rely on a tool available with the macports install, daemondo, which I find is really useful to get traditional services to play nicely with launchd. But of course, user can just modify the installed files to run however they want. All dependent libraries were installed via macports including build tools ( autoconf, automake, etc. )
Added install hook to comment out Xorg and Xvnc blocks automatically, since I believe the only way possible to run on MacOS is through vnc-any.
Made minor enhancement to facilitate how I run it. You have a nice gateway method where you can configure the sesman to act as a gateway for login. However, when I use this with the any-vnc configuration, I could not find a way to configure it because I need the gateway password to be the password the user entered, and the password to be the password for the VNC server. So added the "ask" option for the gateway username and gateway password so they can use the rdp username and password. Running this way, I don't have to rely on VNC with its low security password, and instead can do real authentication. I can also lock down the VNC server to localhost only. My current VNC server of choice is currently OSXvnc-server( now vine server ). I find the performance is quite good.

Fix bad space in LaunchDaemons directory.
Remove use of daemondo.
Fix formatting with astyle.
@mstone2001
Copy link
Author

So, based on the previous comments, I investigated the need for the external daemondo for the launchd plists. And it looks like it is not necessary, so I have simplified the plist files to directly launch the xrdp and xrdp-sesman services.
I also cleaned up trailing whitespace and ran the astyle program to fix the indentation issues.

@alexmyczko
Copy link

is this vnc only or will rdp also work?

@mstone2001
Copy link
Author

is this vnc only or will rdp also work?

This is using xrdp via the vnc method. So you run a vnc server on our Mac( I use Osxvnc, formally vine server ), and configure xrdp to use the local vnc connection. This way, you can use Remote Desktop to connect and authenticate, and lock down the VNC server to localhost only so it's more secure.

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