renamed from defaultPublicRepository to defaultCommunityRegistry

src/main/groovy/io/seqera/wave/auth/RegistryCredentialsProviderImpl.groovy

@@ -113,10 +113,10 @@ class RegistryCredentialsProviderImpl implements RegistryCredentialsProvider {
             throw new IllegalArgumentException("Missing required parameter userId -- Unable to retrieve credentials for container repository '$container'")
 
         // use default credentials for default repositories
-        // NOTE: this requires that 'defaultBuildRepository', 'defaultCacheRepository' and 'defaultPublicRepository' have a unique registry host name
+        // NOTE: this requires that 'defaultBuildRepository', 'defaultCacheRepository' and 'defaultCommunityRegistry' have a unique registry host name
         // that means that for example docker.io/some/repo should not be used otherwise wave credentials could be used in place of user credentials
         // for a repo having the same registry host
-        if( container.sameRegistry(buildConfig.defaultBuildRepository) || container.sameRegistry(buildConfig.defaultCacheRepository) || container.sameRegistry(buildConfig.defaultPublicRepository) )
+        if( container.sameRegistry(buildConfig.defaultBuildRepository) || container.sameRegistry(buildConfig.defaultCacheRepository) || container.sameRegistry(buildConfig.defaultCommunityRegistry) )
             return getDefaultCredentials(container)
 
         return getUserCredentials0(container.registry, identity)

src/main/groovy/io/seqera/wave/configuration/BuildConfig.groovy

@@ -55,7 +55,7 @@ class BuildConfig {
 
     @Nullable
     @Value('${wave.build.public-repo}')
-    String defaultPublicRepository
+    String defaultCommunityRegistry
 
     /**
      * File system path there the dockerfile is save
@@ -120,7 +120,7 @@ class BuildConfig {
                 "singularity-image-amr64=${singularityImageArm64}; " +
                 "default-build-repository=${defaultBuildRepository}; " +
                 "default-cache-repository=${defaultCacheRepository}; " +
-                "default-public-repository=${defaultPublicRepository}; " +
+                "default-public-repository=${defaultCommunityRegistry}; " +
                 "build-workspace=${buildWorkspace}; " +
                 "build-timeout=${defaultTimeout}; " +
                 "build-trusted-timeout=${trustedTimeout}; " +

src/main/groovy/io/seqera/wave/controller/ContainerController.groovy

@@ -176,7 +176,7 @@ class ContainerController {
 
     @PostConstruct
     private void init() {
-        log.info "Wave server url: $serverUrl; allowAnonymous: $allowAnonymous; tower-endpoint-url: $towerEndpointUrl; default-build-repo: $buildConfig.defaultBuildRepository; default-cache-repo: $buildConfig.defaultCacheRepository; default-public-repo: $buildConfig.defaultPublicRepository"
+        log.info "Wave server url: $serverUrl; allowAnonymous: $allowAnonymous; tower-endpoint-url: $towerEndpointUrl; default-build-repo: $buildConfig.defaultBuildRepository; default-cache-repo: $buildConfig.defaultCacheRepository; default-public-repo: $buildConfig.defaultCommunityRegistry"
     }
 
     @Deprecated
@@ -249,7 +249,7 @@ class ContainerController {
         if( req.containerImage && req.freeze && !validationService.isCustomRepo(req.buildRepository) )
             throw new BadRequestException("Attribute `buildRepository` must be specified when using freeze mode [2]")
 
-        if( v2 && req.packages && req.freeze && !validationService.isCustomRepo(req.buildRepository) && !buildConfig.defaultPublicRepository )
+        if( v2 && req.packages && req.freeze && !validationService.isCustomRepo(req.buildRepository) && !buildConfig.defaultCommunityRegistry )
             throw new BadRequestException("Attribute `buildRepository` must be specified when using freeze mode [3]")
 
         if( v2 && req.packages ) {
@@ -295,9 +295,9 @@ class ContainerController {
     protected String targetRepo(String repo, ImageNameStrategy strategy) {
         assert repo, 'Missing default public repository setting'
         // ignore everything that's not a public (community) repo
-        if( !buildConfig.defaultPublicRepository )
+        if( !buildConfig.defaultCommunityRegistry )
             return repo
-        if( !repo.startsWith(buildConfig.defaultPublicRepository) )
+        if( !repo.startsWith(buildConfig.defaultCommunityRegistry) )
             return repo
 
         // check if the repository does use any reserved word
@@ -333,8 +333,8 @@ class ContainerController {
         final condaContent = condaFileFromRequest(req)
         final format = req.formatSingularity() ? SINGULARITY : DOCKER
         final platform = ContainerPlatform.of(req.containerPlatform)
-        final buildRepository = targetRepo( req.buildRepository ?: (req.freeze && buildConfig.defaultPublicRepository
-                ? buildConfig.defaultPublicRepository
+        final buildRepository = targetRepo( req.buildRepository ?: (req.freeze && buildConfig.defaultCommunityRegistry
+                ? buildConfig.defaultCommunityRegistry
                 : buildConfig.defaultBuildRepository), req.nameStrategy)
         final cacheRepository = req.cacheRepository ?: buildConfig.defaultCacheRepository
         final configJson = inspectService.credentialsConfigJson(containerSpec, buildRepository, cacheRepository, identity)
@@ -343,8 +343,8 @@ class ContainerController {
         // use 'imageSuffix' strategy by default for public repo images
         final nameStrategy = req.nameStrategy==null
                 && buildRepository
-                && buildConfig.defaultPublicRepository
-                && buildRepository.startsWith(buildConfig.defaultPublicRepository) ? ImageNameStrategy.imageSuffix : req.nameStrategy
+                && buildConfig.defaultCommunityRegistry
+                && buildRepository.startsWith(buildConfig.defaultCommunityRegistry) ? ImageNameStrategy.imageSuffix : req.nameStrategy
 
         checkContainerSpec(containerSpec)
 

src/main/groovy/io/seqera/wave/service/validation/ValidationServiceImpl.groovy

@@ -107,7 +107,7 @@ class ValidationServiceImpl implements ValidationService {
     boolean isCustomRepo(String repo) {
         if( !repo )
             return false
-        if( buildConfig.defaultPublicRepository && repo.startsWith(buildConfig.defaultPublicRepository) )
+        if( buildConfig.defaultCommunityRegistry && repo.startsWith(buildConfig.defaultCommunityRegistry) )
             return false
         if( buildConfig.defaultBuildRepository && repo.startsWith(buildConfig.defaultBuildRepository) )
             return false

src/test/groovy/io/seqera/wave/controller/ContainerControllerTest.groovy

@@ -626,7 +626,7 @@ class ContainerControllerTest extends Specification {
     @Unroll
     def 'should normalise community repo' () {
         given:
-        def config = new BuildConfig(defaultPublicRepository: PUBLIC, reservedWords: ['build','library'] as Set)
+        def config = new BuildConfig(defaultCommunityRegistry: PUBLIC, reservedWords: ['build','library'] as Set)
         def controller = new ContainerController(buildConfig: config)
         expect:
         controller.targetRepo(REPO, STRATEGY) == EXPECTED
@@ -655,7 +655,7 @@ class ContainerControllerTest extends Specification {
 
     def 'should not allow reserved words' () {
         given:
-        def config = new BuildConfig(defaultPublicRepository: 'foo.com',  reservedWords: ['build','library'] as Set)
+        def config = new BuildConfig(defaultCommunityRegistry: 'foo.com',  reservedWords: ['build','library'] as Set)
         def controller = new ContainerController(buildConfig: config)
 
         when:

src/test/groovy/io/seqera/wave/service/validation/ValidationServiceTest.groovy

@@ -38,7 +38,7 @@ class ValidationServiceTest extends Specification {
     @MockBean(BuildConfig)
     BuildConfig buildConfig() {
         Mock(BuildConfig) {
-            defaultPublicRepository >> 'public.repo.io'
+            defaultCommunityRegistry >> 'public.repo.io'
         }
     }