search

getappmap / appmap-java

AppMap client agent for Java
Other
80 stars 14 forks source link

fix: correct detection of Spark server #266

Closed apotterri closed 5 months ago

apotterri commented 5 months ago

When instrumenting a Jetty server, only add a Spark handler if the app is really a Spark app. A mistake in the code meant that it added the handler whenever running in a Jetty server. This would cause request recording to fail (and maybe cause other problems, as well).

Fixes #265 .

apotterri commented 5 months ago

Hmmm, I think you're right. Given that, it's weird that the Spark tests pass. I'll investigate why that is.

appland-release commented 5 months ago

:tada: This PR is included in version 1.26.5 :tada:

The release is available on:

Your semantic-release bot :package::rocket: