Issue Details (XML | Word | Printable)

Key: MNG-3652
Type: Improvement Improvement
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Brett Porter
Reporter: Brett Porter
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Maven 2

set a user agent for Maven HTTP requests

Created: 03/Jul/08 11:31 PM   Updated: 03/Sep/08 04:59 PM
Component/s: Artifacts and Repositories
Affects Version/s: 2.0.9
Fix Version/s: 2.0.10, 2.1.0-M1

Time Tracking:
Not Specified

Issue Links:
Related
 

Complexity: Intermediate


 Description  « Hide
Apache Maven/2.1-SNAPSHOT

 All   Comments   Work Log   Change History      Sort Order: Ascending order - Click to sort in descending order
Brett Porter added a comment - 04/Jul/08 12:02 AM
ported to the 2.0.x branch

John Casey added a comment - 22/Jul/08 11:13 AM
I revised this for 2.0.10 (needs to be propagated to 2.0.x and 2.1.x branches still), and the revision works with all wagon classes that have a setHttpHeaders method and where the protocol.startsWith( "http" )...it also avoids the need for an improved shade plugin to merge component configurations.

That was in: https://svn.apache.org/repos/asf/maven/components/branches/maven-2.0.10-RC, revId: 678784

Also, I revised the User-Agent integration test to setup a small ServerSocket and record the HTTP headers that were sent from WagonManager.getArtifact(..) directed at that server socket. I then compare this User-Agent header to information collected by the plugin that triggers the WagonManager call, and written out to the touch file. This information includes maven version (since the System property "maven.version" isn't present when the unit test is run directly using the -Dtest=... surefire parameter), java version, O/S name, and O/S version.

The User-Agent string itself has also been improved to be more consistent with the values used by other engines. See:

http://www.user-agents.org/index.shtml?moz

for examples.


John Casey added a comment - 22/Jul/08 11:13 AM
BTW, the updated integration test was committed in revId: 678785

John Casey added a comment - 03/Sep/08 04:59 PM
Adding fix-for for both 2.0.10 and 2.1.0-M1, since 2.1.0-M1 will actually be released first and may not incorporate all of the eventual issue fixes released in 2.0.10.