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

Cleanup plugin security #14311

Closed
wants to merge 2 commits into from

Conversation

Projects
None yet
2 participants
@rmuir
Copy link
Contributor

commented Oct 27, 2015

  • plugin authors can use full policy syntax, including codebase substitution
    properties like core syntax.
  • simplify test logic.
  • move out test-framework permissions to separate file.
Cleanup plugin security
* plugin authors can use full policy syntax, including codebase substitution
  properties like core syntax.
* simplify test logic.
* move out test-framework permissions to separate file.
@s1monw

This comment has been minimized.

Copy link
Contributor

commented Oct 27, 2015

this looks awesome! I love the cleanups related to test framework! LGTM

@rmuir rmuir closed this in eec3c2a Oct 28, 2015

rmuir added a commit that referenced this pull request Oct 28, 2015

Cleanup plugin security
* plugin authors can use full policy syntax, including codebase substitution
  properties like core syntax.
* simplify test logic.
* move out test-framework permissions to separate file.

Closes #14311

Conflicts:
	core/src/main/java/org/elasticsearch/bootstrap/Security.java
	core/src/main/resources/org/elasticsearch/bootstrap/security.policy
	core/src/test/java/org/elasticsearch/bootstrap/BootstrapForTesting.java
	core/src/test/java/org/elasticsearch/bootstrap/MockPluginPolicy.java
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.