Skip to content

Commit

Permalink
Merge pull request #1252 from mateodemuynck/fix/stack-domains
Browse files Browse the repository at this point in the history
fix: expose docker stack services
  • Loading branch information
Siumauricio authored Feb 1, 2025
2 parents f14ed51 + af083ff commit ee2fed0
Showing 1 changed file with 18 additions and 4 deletions.
22 changes: 18 additions & 4 deletions packages/server/src/utils/docker/domain.ts
Original file line number Diff line number Diff line change
Expand Up @@ -203,9 +203,6 @@ export const addDomainToCompose = async (
if (!result?.services?.[serviceName]) {
throw new Error(`The service ${serviceName} not found in the compose`);
}
if (!result.services[serviceName].labels) {
result.services[serviceName].labels = [];
}

const httpLabels = await createDomainLabels(appName, domain, "web");
if (https) {
Expand All @@ -217,7 +214,24 @@ export const addDomainToCompose = async (
httpLabels.push(...httpsLabels);
}

const labels = result.services[serviceName].labels;
let labels: DefinitionsService["labels"] = [];
if (compose.composeType === "docker-compose") {
if (!result.services[serviceName].labels) {
result.services[serviceName].labels = [];
}

labels = result.services[serviceName].labels;
} else {
// Stack Case
if (!result.services[serviceName].deploy) {
result.services[serviceName].deploy = {};
}
if (!result.services[serviceName].deploy.labels) {
result.services[serviceName].deploy.labels = [];
}

labels = result.services[serviceName].deploy.labels;
}

if (Array.isArray(labels)) {
if (!labels.includes("traefik.enable=true")) {
Expand Down

0 comments on commit ee2fed0

Please sign in to comment.