kwin commented on code in PR #1771:
URL: https://github.com/apache/maven-resolver/pull/1771#discussion_r2736019122
##########
maven-resolver-spi/src/main/java/org/eclipse/aether/spi/connector/transport/TransporterFactory.java:
##########
@@ -31,9 +33,39 @@
*/
public interface TransporterFactory {
+ /**
+ * A key for transporter properties.
+ * @see #getProperties()
+ */
+ public interface TransporterPropertyKey {}
+
+ /**
+ * Indicates whether this factory can handle the specified repository
protocol.
+ * Even if {@code true} is returned the factory may still refuse to create
a transporter for the given protocol.
+ *
+ * @param repositoryProtocol The repository protocol to check, may be
{@code null}.
+ * @return {@code true} if this factory can potentially handle the
specified repository protocol, {@code false} otherwise.
+ * @see #newInstance(RepositorySystemSession, RemoteRepository)
+ */
+ default boolean canHandle(String repositoryProtocol) {
Review Comment:
Maybe to ease listing all transporters it would be better to expose a list
of supported protocols (potentially including a wildcard). WDYT @cstamas?
--
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]
