Use of proxy properties in lightweight http wagon is not thread-safe

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.0-beta-6
  • Fix Version/s: 2.0
  • Component/s: wagon-http-lightweight
  • Labels:
    None

Description

The usage of system properties to configure the proxy for the lightweight wagon makes it not thread safe. E.g. as discovered by the Maven IT for MNG-2305, this concurrency issue makes parallel artifact resolution via proxy fail intermittently.

Moving on to Java 1.5+ APIs, it should be feasible to fix this via per-connection proxy settings.

Issue Links

relates to

Bug - A problem which impairs or prevents the functions of the product. WAGON-346 LightWeight http wagon not thread-safe

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Hide
Permalink
Jan Sievers added a comment -

The only workaround we found is to use the commons http wagon implementation

<build>
<extensions>
<extension>
<groupId>org.apache.maven.wagon</groupId>
<artifactId>wagon-http</artifactId>
<version>1.0-beta-7</version>
</extension>
</extensions>
</build>

We also tried reducing the number of threads used to 1 uisng "-Daether.connector.wagon.threads=1" but this did not help.

This problem appears globally for us and changing all poms/parent poms to use a different wagon is not feasible.
Do you have other options for workarounds?

Thanks
Jan

Show
Jan Sievers added a comment - The only workaround we found is to use the commons http wagon implementation <build> <extensions> <extension> <groupId>org.apache.maven.wagon</groupId> <artifactId>wagon-http</artifactId> <version>1.0-beta-7</version> </extension> </extensions> </build> We also tried reducing the number of threads used to 1 uisng "-Daether.connector.wagon.threads=1" but this did not help. This problem appears globally for us and changing all poms/parent poms to use a different wagon is not feasible. Do you have other options for workarounds? Thanks Jan

People

  • Assignee:
    Olivier Lamy
    Reporter:
    Benjamin Bentmann
Vote (0)
Watch (2)

Dates

  • Created:
    Updated:
    Resolved: