Skip to content

Conversation

@liulinC
Copy link
Collaborator

@liulinC liulinC commented Apr 3, 2025

According to https://techcommunity.microsoft.com/blog/coreinfrastructureandsecurityblog/decrypting-the-selection-of-supported-kerberos-encryption-types/1628797 msDS-SupportedEncryptionTypes of machine account help to decide Service Ticket encryption type

Some customer IT teams have strict encryption types limitation in their domains

This commit add winbind_set_machine_account_kerberos_encryption_type and default to false. When enabled, xapi set the machine account encryption types consistent with the samba client

@liulinC liulinC force-pushed the private/linl/etype branch from 04e6799 to 6583745 Compare April 3, 2025 07:56
@liulinC
Copy link
Collaborator Author

liulinC commented Apr 3, 2025

This xapi will require new version of samba.

@liulinC liulinC force-pushed the private/linl/etype branch from 6583745 to ac0ea24 Compare April 3, 2025 09:03
@liulinC liulinC force-pushed the private/linl/etype branch 2 times, most recently from 726c764 to 1753219 Compare April 9, 2025 02:19
@liulinC liulinC force-pushed the private/linl/etype branch from 1753219 to 0d49ea7 Compare April 9, 2025 03:28
Copy link
Contributor

@lindig lindig left a comment

Choose a reason for hiding this comment

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

Nice improvements

According to https://techcommunity.microsoft.com/blog/coreinfrastructureandsecurityblog/decrypting-the-selection-of-supported-kerberos-encryption-types/1628797
msDS-SupportedEncryptionTypes of machine account help to decide
Service Ticket encryption type

Some customer IT teams have strict encryption types limitation in
their domains

This commit add winbind_set_machine_account_kerberos_encryption_type
and default to false. When enabled, xapi set the machine account
encryption types consistent with the samba client

Signed-off-by: Lin Liu <[email protected]>
@liulinC liulinC force-pushed the private/linl/etype branch from 0d49ea7 to cbeb0fc Compare April 10, 2025 01:55
@liulinC liulinC added this pull request to the merge queue Apr 11, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 11, 2025
@liulinC liulinC added this pull request to the merge queue Apr 11, 2025
Merged via the queue into xapi-project:master with commit 3797ffc Apr 11, 2025
17 checks passed
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.

5 participants