cstamas commented on code in PR #1790:
URL: https://github.com/apache/maven-resolver/pull/1790#discussion_r2790034886
##########
maven-resolver-transport-http/src/main/java/org/eclipse/aether/transport/http/HttpTransporter.java:
##########
@@ -358,6 +363,18 @@ final class HttpTransporter extends AbstractTransporter {
builder.setConnectionReuseStrategy(NoConnectionReuseStrategy.INSTANCE);
}
+ if (session.getCache() != null) {
+ String authCacheKey = getClass().getSimpleName() + "-" +
repository.getId() + "-"
+ + StringDigestUtil.sha1(repository.toString());
Review Comment:
presence of repo.id is really just for simpler debugging :smile:
But in general, yes, as your pom may define remote repo named "foo" and then
2 levels deeper a transitive POM may redefine remote repo "foo" with completely
different URL.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
