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

BlockHound integration #1821

Merged
merged 15 commits into from
Mar 16, 2020
Merged

BlockHound integration #1821

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
edc388c
BlockHound integration POC
dkhalanskyjb Feb 3, 2020
ef33b34
Do not shadow ByteBuddy agent
dkhalanskyjb Feb 6, 2020
69f76dc
Fix tests not being run with the right dispatcher
dkhalanskyjb Feb 6, 2020
46bbf2a
Clean the stack traces from BlockHound artifacts
dkhalanskyjb Feb 6, 2020
d1e4798
Intermediate value
dkhalanskyjb Feb 18, 2020
5058081
Improve handling of stacktrace tests on JDK 8
dkhalanskyjb Feb 19, 2020
efb4720
Implement BlockHound integration properly
dkhalanskyjb Feb 19, 2020
6e5fe92
Up the BlockHound version
dkhalanskyjb Feb 19, 2020
7fc4a90
Fixes
dkhalanskyjb Mar 13, 2020
3fc6abc
Improve build configuration of integration tests
dkhalanskyjb Mar 16, 2020
76ab90f
Add an integration test for coroutine debugger java agent
dkhalanskyjb Mar 16, 2020
e16face
Document BlockHound integration better
dkhalanskyjb Mar 16, 2020
15b54c9
Do not exclude the ByteBuddy agent from shading
dkhalanskyjb Mar 16, 2020
e685384
Documentation improvement
qwwdfsad Mar 16, 2020
20cc2ff
Update kotlinx-coroutines-debug/README.md
qwwdfsad Mar 16, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Improve handling of stacktrace tests on JDK 8 
Before, the tests only knew that the `park` native method was moved
to `jdk.internal.misc.Unsafe` from `sun.misc.Unsafe`. However, in
JDK 11, there is no `sun.misc.Unsafe` at all, it seems that it
has been moved completely.

This change is beneficial for this feature set because BlockHound
puts its `$$BlockHound$$_park` method to `sun.misc.Unsafe` or to
`jdk.internal.misc.Unsafe` depending on the JDK version. Also,
judging by BlockHound's code
https://github.com/reactor/BlockHound/blob/091d7b139479b1c41eea59baa23389d673fdf73b/agent/src/main/java/reactor/blockhound/BlockHound.java#L177-L187,
`forkAndExec` had been moved, too, but it is not used in tests.
  • Loading branch information
@dkhalanskyjb
dkhalanskyjb committed Mar 13, 2020
commit 5058081623ee3c087fe31a5cbe76b14df462f34d
2 changes: 1 addition & 1 deletion kotlinx-coroutines-debug/test/StracktraceUtils.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ public fun String.trimStackTrace(): String =
.replace(Regex("#[0-9]+"), "")
.replace(Regex("(?<=\tat )[^\n]*/"), "")
.replace(Regex("\t"), "")
.replace("sun.misc.Unsafe.park", "jdk.internal.misc.Unsafe.park") // JDK8->JDK11
.replace("sun.misc.Unsafe.", "jdk.internal.misc.Unsafe.") // JDK8->JDK11
.applyBackspace()

public fun String.applyBackspace(): String {
Expand Down