Skip to content

Conversation

@fyrnaga
Copy link
Collaborator

@fyrnaga fyrnaga commented Jan 22, 2026

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.

Copy link
Collaborator

@joelplourde4 joelplourde4 left a 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 ?!

@fyrnaga
Copy link
Collaborator Author

fyrnaga commented Jan 22, 2026

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;
Copy link
Collaborator Author

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;
Copy link
Collaborator Author

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;
Copy link
Collaborator Author

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;
Copy link
Collaborator Author

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
Copy link
Collaborator Author

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);
}

Copy link
Collaborator Author

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants