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

Remove Environment.homeFile() #12351

Merged
merged 2 commits into from Aug 3, 2015

Conversation

@rmuir
Copy link
Contributor

commented Jul 21, 2015

Today we grant read+write+delete access to any files underneath the home.
But we have to remove this, if we want to have improved security of files
underneath elasticsearch.

Today we grant read+write+delete access to any files underneath the home.
But we have to remove this, if we want to have improved security of files
underneath elasticsearch.
@rmuir rmuir added the v2.0.0-beta1 label Jul 21, 2015
@rjernst

This comment has been minimized.

Copy link
Member

commented Jul 21, 2015

LGTM

@bleskes

This comment has been minimized.

Copy link
Member

commented Jul 21, 2015

LGTM2

Conflicts:
	core/src/test/java/org/elasticsearch/plugins/PluginManagerTests.java
rmuir added a commit that referenced this pull request Aug 3, 2015
Remove Environment.homeFile()
@rmuir rmuir merged commit 22a73d3 into elastic:master Aug 3, 2015
1 check passed
1 check passed
CLA Commit author is a member of Elasticsearch
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.