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

[MNG-7608] Make native resolver transport the m4 default. #885

Closed
wants to merge 2 commits into from

Conversation

cstamas
Copy link
Member

@cstamas cstamas commented Nov 28, 2022

This immediately cuts in "half" the count of HTTP requests against Maven Central or any major MRM.

Altering the meaning of "default": is now same as NATIVE, still leaving it in place for future, as "default" at some point may again become something different than "native".

Depends on apache/maven-integration-testing#215


https://issues.apache.org/jira/browse/MNG-7608

This immediately cuts in "half" the count of HTTP requests
against Maven Central or any major MRM.

Altering the meaning of "default": is now same as NATIVE, still
leaving it in place for future, as "default" at some point may
again become something different than "native".
@cstamas cstamas self-assigned this Nov 28, 2022
@cstamas
Copy link
Member Author

cstamas commented Nov 28, 2022

Default changed (MavenITmng7470ResolverTransportTest.testResolverTransportDefault), and other 3 ITs expect Wagon to be used without overriding default, out of which 2 uses ancient way of setting UA header (using plexus configuration)

@cstamas cstamas changed the title Make native resolver transport the m4 default. [MNG-7608] Make native resolver transport the m4 default. Nov 28, 2022
@cstamas cstamas marked this pull request as ready for review November 28, 2022 18:43
@gnodet gnodet added this to the 4.0.0-alpha-3 milestone Dec 1, 2022
@cstamas
Copy link
Member Author

cstamas commented Dec 2, 2022

Superseded by #892

@cstamas cstamas closed this Dec 2, 2022
@cstamas cstamas deleted the native-transport-default branch December 2, 2022 07:24
@gnodet gnodet removed this from the 4.0.0-alpha-3 milestone Dec 2, 2022
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