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

[Backport][ipa-4-7] Support Samba 4.9 #2390

Closed
wants to merge 1 commit into from

Conversation

tiran
Copy link
Member

@tiran tiran commented Sep 26, 2018

This PR was opened automatically because PR #2373 was pushed to master and backport to ipa-4-7 is required.

Samba 4.9 became a bit more strict about creating a local NT token and a
failure to resolve or create BUILTIN\Guests group will cause a rejection
of the connection for a successfully authenticated one.

Add a default mapping of the nobody group to BUILTIN\Guests.

BUILTIN\Guests is a special group SID that is added to the NT token for
authenticated users.

For real guests there is 'guest account' option in smb.conf which
defaults to 'nobody' user.

This was implicit behavior before as 'guest account = nobody' by
default would pick up 'nobody' group as well.

Fixes: https://pagure.io/freeipa/issue/7705
@tiran
Copy link
Member Author

tiran commented Sep 26, 2018

PR was ACKed automatically because this is backport of PR #2373. Wait for CI to finish before pushing. In case of questions or problems contact @abbra who is author of the original PR.

@tiran tiran added ack Pull Request approved, can be merged re-run Trigger a new run of PR-CI labels Sep 26, 2018
@freeipa-pr-ci freeipa-pr-ci removed the re-run Trigger a new run of PR-CI label Sep 26, 2018
@tiran tiran added the pushed Pull Request has already been pushed label Sep 26, 2018
@tiran
Copy link
Member Author

tiran commented Sep 26, 2018

ipa-4-7:

@tiran tiran closed this Sep 26, 2018
@tiran tiran deleted the backport_pr2373_ipa-4-7 branch March 29, 2019 09:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged pushed Pull Request has already been pushed
Projects
None yet
3 participants