Col-E / Col-E/InstrumentationServer

InstrumentationHelper#populateExisting is inconsistent in some environments

Open
#5 1 comment 0 reactions 0 assignees View on GitHub
bug
Dominant language
Java
Stars
18
Forks
4
PR merge metrics
No merged PRs in 30d

Description

From xxDark

> `populateExisting` will not work as expected in environments like launchwrapper and all this stuff, better approach would be to ignore VM generated classes for lambdas, proxies, etc and call retransformClasses on all classes (calling retransformClasses on VM-generated classes will cause it to crash because some constants are injected using cpPatches (e.g by InvokerBytecodeGenerator))

Contributor guide

No contributing guide indexed for this repository

Research direction

Start at InstrumentationHelper#populateExisting and inspect how it selects classes for retransformation. Compare its behavior in launchwrapper-like environments with the constraints around VM-generated lambda and proxy classes described in the issue. Done means the inconsistent behavior is resolved without retransformation crashes, but the issue does not name tests or a specific validation path.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
tooling
Issue type
Bug
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.