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.

Sign up for GitHub

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

Jump to bottom

ServiceOperationInvoker#getValidatedServiceAccess inefficient reflection #1106

Open
wants to merge 1 commit into
base: releases/24.2
Choose a base branch
from
Open

ServiceOperationInvoker#getValidatedServiceAccess inefficient reflection #1106

wants to merge 1 commit into from

Commits on Aug 16, 2024

  1. ServiceOperationInvoker#getValidatedServiceAccess inefficient reflection 

    Add lazy initialized cache for public service methods for each queried
class. Additionally fetching all public methods of a class at once
avoids catching NoSuchMethodExceptions when trying to find a method.

This change avoids repeated NoSuchMethodException when checking if
access is allowed for services using interfaces and abstract class in
their hierarchy, where not every super class contains an implementation
of the invoked method.

359668
    @paolobazzi
    paolobazzi committed Aug 16, 2024
    Configuration menu
    Copy the full SHA
    1e5e40f View commit details
    Browse the repository at this point in the history