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

Use commons-codec and commons-logging #4

Merged
merged 1 commit into from
Jul 7, 2024

Conversation

merks
Copy link
Contributor

@merks merks commented Jul 6, 2024

@nitind nitind merged commit 35e436a into eclipse-webservices:master Jul 7, 2024
1 check passed
@nitind
Copy link
Contributor

nitind commented Jul 7, 2024

Thanks, Ed!

@merks merks deleted the pr-commons-logging-codec branch July 7, 2024 15:22
@nitind
Copy link
Contributor

nitind commented Jul 7, 2024

@merks the version of org.apache.httpcomponents.httpclient we're pulling in conflicts by exporting the old bundle and won't build with this change https://ci.eclipse.org/webtools/view/webtools_CI/job/webtools-webservices_master/2704/:

Bundle was not resolved because of a uses constraint violation.
  org.apache.felix.resolver.reason.ReasonException: Uses constraint violation. Unable to resolve resource org.eclipse.wst.wsi [osgi.identity; type="osgi.bundle"; version:Version="1.1.501.qualifier"; osgi.identity="org.eclipse.wst.wsi"; singleton:="true"] because it is exposed to package 'org.apache.commons.logging' from resources org.apache.commons.commons-logging [osgi.identity; type="osgi.bundle"; version:Version="1.3.3"; osgi.identity="org.apache.commons.commons-logging"] and org.apache.commons.logging [osgi.identity; type="osgi.bundle"; version:Version="1.2.0.v20180409-1502"; osgi.identity="org.apache.commons.logging"] via two dependency chains.

Chain 1:
  org.eclipse.wst.wsi [osgi.identity; type="osgi.bundle"; version:Version="1.1.501.qualifier"; osgi.identity="org.eclipse.wst.wsi"; singleton:="true"]
    require: (osgi.wiring.bundle=org.apache.commons.commons-logging)
     |
    provide: osgi.wiring.bundle: org.apache.commons.commons-logging
  org.apache.commons.commons-logging [osgi.identity; type="osgi.bundle"; version:Version="1.3.3"; osgi.identity="org.apache.commons.commons-logging"]

Chain 2:
  org.eclipse.wst.wsi [osgi.identity; type="osgi.bundle"; version:Version="1.1.501.qualifier"; osgi.identity="org.eclipse.wst.wsi"; singleton:="true"]
    import: (osgi.wiring.package=org.apache.http.impl.conn)
     |
    export: osgi.wiring.package=org.apache.http.impl.conn; uses:=org.apache.commons.logging
  org.apache.httpcomponents.httpclient [osgi.identity; type="osgi.bundle"; version:Version="4.5.14"; osgi.identity="org.apache.httpcomponents.httpclient"]
    import: (&(osgi.wiring.package=org.apache.commons.logging)(&(version>=1.1.0)(!(version>=1.3.0))))
     |
    export: osgi.wiring.package: org.apache.commons.logging
  org.apache.commons.logging [osgi.identity; type="osgi.bundle"; version:Version="1.2.0.v20180409-1502"; osgi.identity="org.apache.commons.logging"]`

@nitind
Copy link
Contributor

nitind commented Jul 8, 2024

I'm working on porting that class to use the v5 client, so that we're not the ones requiring v4 to stick around. We'll see how that goes.

@merks
Copy link
Contributor Author

merks commented Jul 8, 2024

That sounds good.

This bundle has a very restrictive version range dependency on the org.apache.commons.logging package:

[ERROR] Missing requirement: org.apache.httpcomponents.httpasyncclient 4.1.5 requires 'java.package; org.apache.commons.logging [1.1.0,1.3.0)' but it could not be found

That bundle provides these packages:

    <provided namespace='java.package' name='org.apache.http.nio.client' version='4.1.5'/>
    <provided namespace='java.package' name='org.apache.http.nio.client.methods' version='4.1.5'/>
    <provided namespace='java.package' name='org.apache.http.nio.client.util' version='4.1.5'/>
    <provided namespace='java.package' name='org.apache.http.nio.conn' version='4.1.5'/>
    <provided namespace='java.package' name='org.apache.http.nio.conn.scheme' version='4.1.5'/>
    <provided namespace='java.package' name='org.apache.http.nio.conn.ssl' version='4.1.5'/>
    <provided namespace='java.package' name='org.apache.http.impl.nio.client' version='4.1.5'/>
    <provided namespace='java.package' name='org.apache.http.impl.nio.conn' version='4.1.5'/>

@nitind
Copy link
Contributor

nitind commented Jul 8, 2024

The wsi bundle was using the v4 http client bundle to validate headers, and I believe it'll now work against the v5 http core bundle. No orbit changes needed.

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.

2 participants