search

getappmap / appmap-java

AppMap client agent for Java
Other
82 stars 16 forks source link

fix: don't depend on spring-web #188

Closed apotterri closed 1 year ago

apotterri commented 1 year ago

Remove the dependency on org.springframework:spring-web, it should have been removed as part of the changes to implement request recording.

Its inclusion prevents hooking of SpringServletContainerInitializer.onStartup, which keeps request recording doesn't work for a Spring app deployed as a WAR. So, these changes also add a test for such an app.

appland-release commented 1 year ago

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

The release is available on:

Your semantic-release bot :package::rocket: