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 Deprecated Script Settings #24756
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, left real comment
import java.util.Set; | ||
import java.util.function.Function; | ||
|
||
public class ScriptSecurity { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't like this name, this isn't really about security, it is just a feature of scripting in ES to disable certain contexts/types. And I think this is small enough it could just be part of ScriptService which is the only place it is used?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Moved.
} | ||
} | ||
|
||
if (!found) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: == false
please
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed.
@elasticmachine please test this |
1 similar comment
@elasticmachine please test this |
Removes all fine grained settings for script types, contexts, and engines.
Script settings will now be specified as defined in #24532.