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

sesman: move auth_start_session, auth_stop_session, and auth_end to c… #806

Merged
merged 1 commit into from
Jul 12, 2017

Conversation

jsorg71
Copy link
Contributor

@jsorg71 jsorg71 commented Jul 10, 2017

…hild process

@jsorg71
Copy link
Contributor Author

jsorg71 commented Jul 10, 2017

This PR should replace #779

@jsorg71 jsorg71 mentioned this pull request Jul 10, 2017
@metalefty metalefty added this to the v0.9.3 milestone Jul 10, 2017
@metalefty
Copy link
Member

Tested on FreeBSD, I confirmed that this PR breaks nothing for FreeBSD.
If this solves systemd sessions issue, I'm OK with merging this.

@moobyfr
Copy link
Contributor

moobyfr commented Jul 10, 2017

Tested on linux jessie + ldap/krb5, the users have the pam settings applied, so not disturbing the CVE-2017-6967 fix. And it fix the original issue #778 about systemd

@metalefty
Copy link
Member

Let's merge and prepare release.

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.

3 participants