-
Notifications
You must be signed in to change notification settings - Fork 0
MC-30907: Fixed issue with renamed #11
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
base: devel
Are you sure you want to change the base?
Conversation
joelplourde4
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Where does this code from ?!
This code is just a copy of the jersey connector in the source already, but with the package jakarta.ws instead of javax.ws. |
| @@ -0,0 +1,140 @@ | |||
| package org.openstack4j.connectors.jersey2; | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
copied over from connectors/jersey2 and changing javax.ws to jakarta.ws
| @@ -0,0 +1,156 @@ | |||
| package org.openstack4j.connectors.jersey2; | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
copied over from connectors/jersey2 and changing javax.ws to jakarta.ws
| @@ -0,0 +1,77 @@ | |||
| package org.openstack4j.connectors.jersey2; | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
copied over from connectors/jersey2 and changing javax.ws to jakarta.ws
| @@ -0,0 +1,125 @@ | |||
| package org.openstack4j.connectors.jersey2; | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
copied over from connectors/jersey2 and changing javax.ws to jakarta.ws
| @@ -0,0 +1 @@ | |||
| org.openstack4j.connectors.jersey2.HttpExecutorServiceImpl | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
copied over from connectors/jersey2
| } else { | ||
| access = access.applyContext(info.endpoint, (org.openstack4j.openstack.identity.v2.domain.TokenAuth) auth); | ||
| } | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did no change. Intellij did.
PR description
This is the 3 attempt to fix Swift Connection that is failing. We used to use Jersey, but with the spring boot migration that is now using Jakarta instead of Javax packages. It failed.
We tried to use OkHttp, but it triggered connection error to often closed by GCP.
We switch to HttpClient, but we cannot rename files since there is an error because of Content-Length already set.
So this PR is to create a new jersey client using Jakarta implementation instead of javax.