Commit 36109cfb authored by Federico Falconieri's avatar Federico Falconieri
Browse files

Merge branch 'hoverht-master-patch-53446' into 'master' 

fix: only allow valid docker tag names

Closes #45

See merge request ci/templates!19
parents a5e3be01 44441be1

docker/kaniko-template.yml

+5 −0
Original line number Diff line number Diff line
@@ -17,6 +17,11 @@ 
    name: gcr.io/kaniko-project/executor:debug

    entrypoint: [""]

  script:

    - |

      if [ "${CI_COMMIT_REF_NAME}" != "$(echo ${CI_COMMIT_REF_NAME} | sed 's/[^a-zA-Z0-9._]/-/g')" ]; then

        echo "Branch names must follow docker tag rules. Only use ASCII (lower or upper case), digits, underscores, periods and dashes. Your current branch name is '${CI_COMMIT_REF_NAME}'. A suggestion is '$(echo ${CI_COMMIT_REF_NAME} | sed 's/[^a-zA-Z0-9._]/-/g')'"

        exit 1

      fi

    - echo "{\"auths\":{\"${CI_REGISTRY}\":{\"username\":\"${CI_REGISTRY_USER}\",\"password\":\"${CI_REGISTRY_PASSWORD}\"}}}" > /kaniko/.docker/config.json

    - |

      if [ -z "${DESTINATIONS}" ]; then