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

Detect Synchronized Methods #402

Open
jonathannaguin opened this issue Mar 12, 2024 · 1 comment
Open

Detect Synchronized Methods #402

jonathannaguin opened this issue Mar 12, 2024 · 1 comment
Assignees
Labels
type/enhancement A general enhancement

Comments

@jonathannaguin
Copy link

jonathannaguin commented Mar 12, 2024

Synchronized methods should be detected by BlockHound as blocking operations.

Motivation

A library you import on a Reactor project could make use of Synchronized method that will impact the performance of your project due to the nature of the construct.

Desired solution

BlockHound could detect those synchronized methods and/or synchronized statements and throw an exception when used.

Considered alternatives

Code inspection.

Additional context

Info from the Oracle webpage: https://docs.oracle.com/javase/tutorial/essential/concurrency/syncmeth.html.

I am not sure about the feasibility of this check but thought of raising it here.

@jonathannaguin jonathannaguin added the type/enhancement A general enhancement label Mar 12, 2024
@pderop pderop self-assigned this Mar 12, 2024
@pderop
Copy link
Member

pderop commented Mar 12, 2024

Hi @jonathannaguin ,

From my sandbox, I have already conducted a few successful experiments using ASM. Give me some time, and I'll see if I can prepare something.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/enhancement A general enhancement
Projects
None yet
Development

No branches or pull requests

2 participants