Skip to content

Update CI_DEPENDENCY_PROXY_SERVER port value

Steve Abrams requested to merge sabrams/fix-ci-dep-proxy-variables into master

What does this MR do?

Updates the Dependency Proxy predefined environment variables to only include a port if it is non-standard (not 80 or 443).

Including the port was causing problems because users would log into the Dependency proxy using the variable, which would include the port, then pull an image without including the port, leading to an error.

Screenshots (strongly suggested)

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • [-] Label as security and @ mention @gitlab-com/gl-security/appsec
  • [-] The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • [-] Security reports checked/validated by a reviewer from the AppSec team
Edited by Steve Abrams

Merge request reports