cstamas commented on code in PR #228:
URL: https://github.com/apache/maven-resolver/pull/228#discussion_r1044593073
##########
maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/DefaultRepositoryConnectorProvider.java:
##########
@@ -109,6 +109,20 @@ public RepositoryConnector newRepositoryConnector(
RepositorySystemSession sessi
throws NoRepositoryConnectorException
{
requireNonNull( repository, "remote repository cannot be null" );
+
+ if ( repository.isBlocked() )
+ {
+ if ( repository.getMirroredRepositories().isEmpty() )
+ {
+ throw new NoRepositoryConnectorException( repository, "Blocked
repository: " + repository );
Review Comment:
Yes, as connector provider provides connector _for one given remote repo_
(is param on method), so it will happily provide connector instance for other
non-blocked remote repositories....
--
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: issues-unsubscr...@maven.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
