Handle empty registry property in input

packages/docker/src/dockerCommands/container.ts

@@ -3,6 +3,7 @@ import * as fs from 'fs'
 import {
   ContainerInfo,
   JobContainerInfo,
+  Registry,
   RunContainerStepArgs,
   ServiceContainerInfo,
   StepContainerInfo
@@ -266,19 +267,19 @@ export async function containerPorts(id: string): Promise<string[]> {
   return portMappings.split('\n')
 }
 
-export async function registryLogin(args): Promise<string> {
+export async function registryLogin(registry?: Registry): Promise<string> {
-  if (!args.registry) {
+  if (!registry) {
     return ''
   }
   const credentials = {
-    username: args.registry.username,
+    username: registry.username,
-    password: args.registry.password
+    password: registry.password
   }
 
   const configLocation = `${env.RUNNER_TEMP}/.docker_${uuidv4()}`
   fs.mkdirSync(configLocation)
   try {
-    await dockerLogin(configLocation, args.registry.serverUrl, credentials)
+    await dockerLogin(configLocation, registry.serverUrl, credentials)
   } catch (error) {
     fs.rmdirSync(configLocation, { recursive: true })
     throw error
@@ -296,7 +297,7 @@ export async function registryLogout(configLocation: string): Promise<void> {
 async function dockerLogin(
   configLocation: string,
   registry: string,
-  credentials: { username: string; password: string }
+  credentials: { username?: string; password?: string }
 ): Promise<void> {
   const credentialsArgs =
     credentials.username && credentials.password

packages/docker/src/hooks/run-container-step.ts

@@ -15,7 +15,7 @@ export async function runContainerStep(
 ): Promise<void> {
   const tag = generateBuildTag() // for docker build
   if (args.image) {
-    const configLocation = await registryLogin(args)
+    const configLocation = await registryLogin(args.registry)
     try {
       await containerPull(args.image, configLocation)
     } finally {