-
Bug
-
Resolution: Unresolved
-
Minor
-
None
Today, to define DockerHub credentials, you have to define an empty url in docker.withRegistry('', 'my-dockerhub-credentials').
It supports an empty string '' but not null.
It would be nice to also support null: docker.withRegistry(null, 'my-dockerhub-credentials') and maybe to add a method such as docker.withRegistryCredentials(credentials).
This code needs a null check. Also documentation needs to mention that null is allowed to refer to DockerHub.