Skip to content

Commit ce85d76

Browse files
committed
only require credentials for non-ecr registries
1 parent 3840d95 commit ce85d76

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

apps/webapp/app/v3/services/finalizeDeploymentV2.server.ts

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,11 @@ export class FinalizeDeploymentV2Service extends BaseService {
8686
const isV4Deployment = deployment.type === "MANAGED";
8787
const registryConfig = getRegistryConfig(isV4Deployment);
8888

89-
if (!registryConfig.host || !registryConfig.username || !registryConfig.password) {
89+
// For non-ECR registries, username and password are required upfront
90+
if (
91+
!isEcrRegistry(registryConfig.host) &&
92+
(!registryConfig.username || !registryConfig.password)
93+
) {
9094
throw new ServiceValidationError("Missing deployment registry credentials");
9195
}
9296

0 commit comments

Comments
 (0)