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

Fix support for configuration files, especially with relation to --protected-mode #61

Merged
merged 1 commit into from
May 11, 2016

Conversation

tianon
Copy link
Contributor

@tianon tianon commented May 10, 2016

Fixes #60 (thanks for the detailed report, @Luzifer!)

@tianon
Copy link
Contributor Author

tianon commented May 10, 2016

Tomorrow I'll look at expanding our test suite to include a config file test, too. 👍

@yosifkit
Copy link
Contributor

LGTM

@yosifkit yosifkit merged commit 0066d38 into redis:master May 11, 2016
@yosifkit yosifkit deleted the conf branch May 11, 2016 19:24
tianon added a commit to infosiftr/stackbrew that referenced this pull request May 11, 2016
@antonioborondo
Copy link

I fear that this fix does not resolve the problem explained in #60 and I am still able to reproduce it #67.

RichardScothern pushed a commit to RichardScothern/official-images that referenced this pull request Jun 14, 2016
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