Skip to content

Conversation

@necouchman
Copy link
Contributor

Adds the required parameters to configure the FreeRDP library to force Kerberos and configure a couple of the parameters related to that, if required. I have tested the changes against both servers with NTLM still enabled and against one where NTLM is disabled, and it seems to work, but others should feel free to test.

I do need one bit of help with this - I have not added the detection of the Kerberos support to the configure.ac file, yet, as I'm struggling to figure out how to do that. In my mind, it needs 2-3 checks:

  • Check if FreeRDP contains support for the FreeRDP_AuthenticationPackageList setting. This is probably the biggest thing I'm struggling with, as this setting is part of an enum in one of the FreeRDP header files, but I'm having trouble finding any guidance on how to generate an autoconf check for an enum member? Any hints on the best way to do that would be appreciated.
  • Check if FreeRDP is built with Kerberos support. There's a flag in the FreeRDP buildconfig.h file, within the constant FREERDP_BUILD_CONFIG that shows WITH_KRB5=ON, but, again, I'm struggling with how to get autoconf to check for this, or identify a sane method or member to use with one of the other autoconf checks.

@necouchman necouchman force-pushed the working/rdp-kerberos branch from 08026b5 to 5c1032c Compare May 3, 2025 12:28

/**
* The authentication package to use based on the underlying FreeRDP support
* for alternatives to NTML. Currently FreeRDP2 only supports NTLM, while
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a typo (NTML > NTLM)

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