wu-sheng opened a new pull request, #813:
URL: https://github.com/apache/skywalking-java/pull/813

   ### Add agent plugins for Jetty 12 and Struts 7 (Jakarta)
   
   Two new plugins for Jakarta-era frameworks that the existing plugins cannot 
instrument:
   
   **Jetty 12 (`jetty-server-12.x`)** — Jetty 12 removed the concrete 
`HttpChannel` (whose `handle()` the merged `jetty-server` plugin enhances) and 
moved request handling to the async, namespace-neutral core API 
`org.eclipse.jetty.server.Server#handle(Request, Response, Callback)`. The 
plugin enhances it and records an async entry span, finished with the response 
status when the wrapped `Callback` completes (with 404 / 500 fallbacks for the 
not-handled and exception paths). Covers Jetty 12.0.x and 12.1.x.
   
   **Struts 7 (`struts2-7.x`)** — Struts 7 relocated `DefaultActionInvocation` 
from `com.opensymphony.xwork2` to `org.apache.struts2` and switched to 
`jakarta.servlet`, so the `struts2-2.x` plugin is inert on it. The new plugin 
enhances the relocated class, in a distinct package (`...plugin.struts2.v7`) to 
avoid an FQN collision with `struts2-2.x`.
   
   - [x] Add a test case for the new plugin: `jetty-12.x-scenario` (embedded 
Jetty 12, versions 12.0.36 / 12.1.10) and `struts2.7-scenario` (Jakarta WAR on 
Tomcat 10.1, versions 7.0.3 / 7.1.1 / 7.2.1), each wired into the JDK-17 CI 
group.
   - [x] Add a component id in `component-libraries.yml`: not needed — both 
reuse existing components (JettyServer / Struts2).
   - [x] Add a logo in the UI repo: not needed — reuse existing framework logos.
   
   - [x] Update the [`CHANGES` 
log](https://github.com/apache/skywalking-java/blob/main/CHANGES.md).
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to