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

*[Sessions]: Now session is accessible if process is ran by root. #1362

Closed
wants to merge 1 commit into
base: PHP-5.5.27
from

Conversation

2 participants
@EvgeniySpinov
Contributor

EvgeniySpinov commented Jun 24, 2015

Fixing bug: https://bugs.php.net/bug.php?id=69582

Use case: session is initiated by Apache with non-root permissions. Later on, session could be accessed by backend process, which runs under root, but needs to communicate with same session data as Apache process.

Didn't work before this commit: session file is owned by non-root, process owner != file owner, so this triggered file handle closure.

@laruence

This comment has been minimized.

Show comment
Hide comment
@laruence

laruence Jun 24, 2015

Member

5.5.27 branch is a release branch, you only needs make a PR against 5.5 branch... thanks

Member

laruence commented Jun 24, 2015

5.5.27 branch is a release branch, you only needs make a PR against 5.5 branch... thanks

@laruence laruence closed this Jun 24, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment