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

Entitlements tools: public callers finder #116257

Merged
merged 17 commits into from
Dec 13, 2024

Conversation

ldematte
Copy link
Contributor

@ldematte ldematte commented Nov 5, 2024

This PR introduces a supporting tool for entitlements.
The tool scans the JDK on which it is running. It takes a list of methods (compatible with the output of the securitymanager-scanner tool), and looks for the "public surface" of these methods (i.e. any class/method accessible from regular Java code that calls into the original list, directly or transitively).

It acts basically as a recursive "Find Usages" in Intellij, stopping at the first fully accessible point (public method on a public class).

See the included README for more details.

@ldematte ldematte added >non-issue :Core/Infra/Core Core issues without another label labels Nov 5, 2024
@ldematte ldematte requested review from jdconrad and a team November 5, 2024 18:14
@ldematte ldematte marked this pull request as ready for review November 5, 2024 18:14
@elasticsearchmachine elasticsearchmachine added the Team:Core/Infra Meta label for core/infra team label Nov 5, 2024
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (Team:Core/Infra)

@ldematte ldematte requested a review from prdoyle November 5, 2024 18:16
@elastic elastic deleted a comment from elasticmachine Nov 6, 2024
@prdoyle prdoyle merged commit 5411b93 into elastic:main Dec 13, 2024
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Core/Infra/Core Core issues without another label >non-issue Team:Core/Infra Meta label for core/infra team v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants