kwin commented on issue #1770: URL: https://github.com/apache/maven-resolver/issues/1770#issuecomment-3810123859
Problem right now is that `TransporterFactory.newInstance` always takes a `RemoteRepository` to check if it should be used. It fails with an exception if it doesn't support the given repository url. Probably this should be refactored so that a more targeted lookup can be performed (also in https://github.com/apache/maven-resolver/blob/ec52693dc400578dd03f731c88c58807cca6a148/maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/DefaultTransporterProvider.java#L104). Maybe adding another method `boolean canHandle(String protocol)` (by default always returning true) to `TransporterFactory` in combination with something like `String getDescription()` (returning some explanation on which library is used etc). would be reasonable. -- 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]
