You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ozone.apache.org by "echonesis (via GitHub)" <gi...@apache.org> on 2026/07/23 16:34:54 UTC
[PR] HDDS-15934. Remove usage of internal jersey internal InjectionManager [ozone]
echonesis opened a new pull request, #10853:
URL: https://github.com/apache/ozone/pull/10853
## What changes were proposed in this pull request?
Recon directly uses Jersey's internal `InjectionManager` API to obtain the HK2 `ServiceLocator` when initializing the Guice-to-HK2 bridge. Depending on an internal Jersey API makes this integration sensitive to Jersey implementation changes.
This PR updates `GuiceResourceConfig` to receive the `ServiceLocator` and `ServletContext` through standard dependency injection. The Guice injector is then bridged to HK2 during resource configuration initialization, without registering a container lifecycle listener or accessing Jersey's internal injection API.
The explicit `jersey-common` dependency is also removed because it is no longer required.
Generated-by: Codex (GPT-5)
## What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-15934
## How was this patch tested?
GitHub Actions CI: https://github.com/echonesis/ozone/actions/runs/30013948598
--
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: issues-unsubscribe@ozone.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org
Re: [PR] HDDS-15934. Remove usage of internal jersey internal InjectionManager [ozone]
Posted by "echonesis (via GitHub)" <gi...@apache.org>.
echonesis commented on PR #10853:
URL: https://github.com/apache/ozone/pull/10853#issuecomment-5087717998
Thanks @chungen0126 for the review and merge.
--
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: issues-unsubscribe@ozone.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org
Re: [PR] HDDS-15934. Remove usage of internal jersey internal InjectionManager [ozone]
Posted by "chungen0126 (via GitHub)" <gi...@apache.org>.
chungen0126 commented on PR #10853:
URL: https://github.com/apache/ozone/pull/10853#issuecomment-5086468084
Merged. Thanks @echonesis for working on this.
--
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: issues-unsubscribe@ozone.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org
Re: [PR] HDDS-15934. Remove usage of internal jersey internal InjectionManager [ozone]
Posted by "chungen0126 (via GitHub)" <gi...@apache.org>.
chungen0126 merged PR #10853:
URL: https://github.com/apache/ozone/pull/10853
--
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: issues-unsubscribe@ozone.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org