René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Remove wrapping from servlet instrumentation ","articleBody":"Wrapping request/response types in servlet can cause issues with casting to specific types.\r\nWe haven't seen any issue in this repo, which can be a result of not wide exposure and lack of tests on different servlet containers.\r\n\r\nOTEL had a wrapping in place for a short period of time and it resulted in issues  https://github.com/open-telemetry/opentelemetry-java-instrumentation/issues/1096. Therefore we should consider removing wrapping and directly instrument streams associated with the request like we do in other instrumentations. \r\n\r\nPart of this task should be performance testing if the stream instrumentation does not add latency/CPU cycles.\r\n\r\n","author":{"url":"https://github.com/pavolloffay","@type":"Person","name":"pavolloffay"},"datePublished":"2021-01-11T08:34:53.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":0},"url":"https://github.com/222/javaagent/issues/222"}