-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
BlockHound false positive in kotlin.jvm.internal.Reflection.renderLambdaToString #4128
Comments
The error complains about our reflect library and we (coroutines) are unlikely to ship BH rules for anything but coroutines themselves. Would it work for you to add a rule in your project for the whole Kotlin reflect package? |
We already do: kotlinx.coroutines/kotlinx-coroutines-debug/src/CoroutinesBlockHoundIntegration.kt Lines 173 to 183 in cd696d3
A bit scary, given that reflection can execute arbitrary user-supplied code. |
Surely a balancing issue, just thought to let you know because some reflection allowance was already there.
Sure, no big deal. [Edit: Not the whole package, but selected reflection functions.]
Agreed. |
kotlinx-coroutines-debug:1.8.0
This one appeared once when running Kotest in debug mode with
env KOTEST_DEBUG=true ./gradlew ...
:Might be worth adding an allowance to
BlockHound.Builder.allowBlockingCallsInReflectionImpl()
at whatever level is deemed appropriate.The text was updated successfully, but these errors were encountered: