Skip to content

Add validation for docker tag names in job.image.name and job.service[].name.

What does this MR do?

This merge request implements docker docker image name validation through a regex: https://regex101.com/r/HaqLc3/3 - you can check the regex match test suite at https://regex101.com/r/HaqLc3/3/tests

The regex is based on the docker image tag spec as available here

This MR fixes one part of #220915 on the server side

Screenshots

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Risks:

  • This is a slightly complicated regex, but with a limited amount of backtracking as far as I can see.
  • (some) additional performance is required for validating ci config files.

Additional tests to check the new validator can be added.

Security

N/A

Edited by Fabio Pitino

Merge request reports