cerc-io/webapp-deployment-status-api
15
0
Fork 1
Code Issues 11 Pull Requests 3 Packages Projects Releases 1 Wiki Activity

[WIP] Handle multiple domains in records in status API #26

Draft
prathamesh wants to merge 1 commits from deep-stack/webapp-deployment-status-api:pm-handle-multiple-dns into main
pull from: deep-stack/webapp-deployment-status-api:pm-handle-multiple-dns
merge into: cerc-io:main
Conversation 0 Commits 1 Files Changed 1 +3 -1
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/deployments.ts
View File

@ -159,7 +159,9 @@ export class RegHelper {
} }
status.app = r.attributes.application; status.app = r.attributes.application;
const hostname = r.attributes.dns ?? generateHostnameForApp(app);
const dnsList: string[] = r.attributes.dns ? r.attributes.dns.split(",") : [];
const hostname = dnsList.length ? dnsList[dnsList.length - 1] : generateHostnameForApp(app);
if (deploymentsByRequest.has(r.id)) { if (deploymentsByRequest.has(r.id)) {
const deployment = deploymentsByRequest.get(r.id); const deployment = deploymentsByRequest.get(r.id);