Maven 2 & 3

set a user agent for Maven HTTP requests

Details

  • Complexity:
    Intermediate
  • Number of attachments :
    0

Description

Apache Maven/2.1-SNAPSHOT

Issue Links

Activity

Hide
Brett Porter added a comment -

ported to the 2.0.x branch

Show
Brett Porter added a comment - ported to the 2.0.x branch
Hide
John Casey added a comment -

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.

Show
John Casey added a comment - 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.
Hide
John Casey added a comment -

BTW, the updated integration test was committed in revId: 678785

Show
John Casey added a comment - BTW, the updated integration test was committed in revId: 678785
Hide
John Casey added a comment -

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.

Show
John Casey added a comment - 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.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: