Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ProxyStore Jersey dependency issues #2868

Closed
GCHQDev404 opened this issue Jan 23, 2023 · 1 comment · Fixed by #2869
Closed

ProxyStore Jersey dependency issues #2868

GCHQDev404 opened this issue Jan 23, 2023 · 1 comment · Fixed by #2869
Assignees
Labels
bug Confirmed or suspected bug proxy-store Specific to/touches the proxy-store module

Comments

@GCHQDev404
Copy link
Contributor

Describe the bug
Proxy store is unable to connect across JVMs and throws a IllegalStateException: InjectionManagerFactory not found.
Likely due to dependency exclusion issue with Jersey.

@GCHQDev404 GCHQDev404 added the bug Confirmed or suspected bug label Jan 23, 2023
@GCHQDeveloper314 GCHQDeveloper314 self-assigned this Jan 23, 2023
@GCHQDeveloper314 GCHQDeveloper314 linked a pull request Jan 23, 2023 that will close this issue
GCHQDev404 added a commit that referenced this issue Jan 23, 2023
…ency-issues' into gh-2866-FederatedStore-add-port-option-to-realfederatedstore-example
GCHQDev404 added a commit that referenced this issue Jan 23, 2023
…ency-issues' into gh-2866-FederatedStore-add-port-option-to-realfederatedstore-example
@t92549 t92549 added this to the v2.0.0-alpha-0.5 milestone Jan 23, 2023
t92549 pushed a commit that referenced this issue Jan 23, 2023
* Remove outdated jersey exclusions and manage versions instead
The exclusions had no effect as the jersey package name had changed. The newer versions can't be excluded as they are required.

* Remove redundent version specification
These versions are now managed across the whole project

* Remove duplicate dependencyManagement
No longer required as now managed across the whole project

* Update Copyright headers

* Move jersey-hk2 to also be a runtime dependency
Appears to be required at runtime by ProxyStore
@t92549
Copy link
Contributor

t92549 commented Jan 23, 2023

Closed by #2869

@t92549 t92549 closed this as completed Jan 23, 2023
GCHQDev404 pushed a commit that referenced this issue Feb 3, 2023
* Remove outdated jersey exclusions and manage versions instead
The exclusions had no effect as the jersey package name had changed. The newer versions can't be excluded as they are required.

* Remove redundent version specification
These versions are now managed across the whole project

* Remove duplicate dependencyManagement
No longer required as now managed across the whole project

* Update Copyright headers

* Move jersey-hk2 to also be a runtime dependency
Appears to be required at runtime by ProxyStore
@t92549 t92549 added the proxy-store Specific to/touches the proxy-store module label Feb 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Confirmed or suspected bug proxy-store Specific to/touches the proxy-store module
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants