Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Split comma-joined container names before exec

App installs can legitimately hold multiple container names: agent/app/service/app_utils.go persists strings.Join(containerNames, ","), and the old terminal dialog split that value before execing. This replacement forwards row.containerName unchanged into containerid=..., so an agent created from a multi-service custom compose will try docker exec foo,bar ..., which Docker rejects and leaves the terminal unusable.

Useful? React with 👍 / 👎.