From ab0125f4cdbd7cf97cb13b02186580fb3820d084 Mon Sep 17 00:00:00 2001 From: Paolo Di Tommaso Date: Sun, 22 Sep 2024 16:51:42 +0200 Subject: [PATCH] Improve missing repo credentials Signed-off-by: Paolo Di Tommaso --- .../inspect/ContainerInspectServiceImpl.groovy | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/src/main/groovy/io/seqera/wave/service/inspect/ContainerInspectServiceImpl.groovy b/src/main/groovy/io/seqera/wave/service/inspect/ContainerInspectServiceImpl.groovy index 03b4e5093..be142ccf0 100644 --- a/src/main/groovy/io/seqera/wave/service/inspect/ContainerInspectServiceImpl.groovy +++ b/src/main/groovy/io/seqera/wave/service/inspect/ContainerInspectServiceImpl.groovy @@ -93,10 +93,13 @@ class ContainerInspectServiceImpl implements ContainerInspectService { if( cacheRepo ) repos.add(cacheRepo) final result = credsJson(repos, identity) - if( buildRepo && result && !result.contains(host0(buildRepo)) ) - throw new BadRequestException("Missing credentials for target build repository: $buildRepo") - if( cacheRepo && result && !result.contains(host0(cacheRepo)) ) - throw new BadRequestException("Missing credentials for target cache repository: $buildRepo") + final msg = identity + ? "Make sure you have provided a corresponding container credentials record in your Seqera Platform account" + : "Make sure you have provided your Seqera Platform token (aka Tower access token) in your request" + if( buildRepo && (!result || !result.contains(host0(buildRepo))) ) + throw new BadRequestException("Missing credentials repository: $buildRepo - $msg") + if( cacheRepo && (!result || !result.contains(host0(cacheRepo))) ) + throw new BadRequestException("Missing credentials repository: $buildRepo - $msg") return result }