fix/dev-sso-private-network-name #57

Merged

longjacksonle merged 2 commits from fix/dev-sso-private-network-name into main

2026-06-04 04:07:29 +02:00

longjacksonle commented

2026-06-04 04:07:26 +02:00

Owner

No description provided.

longjacksonle added 2 commits

2026-06-04 04:07:26 +02:00

fix(dev-sso): match overlay private network name to base per-user name 0aaa5f2d9e

The base compose.dev.yml names the private network `dev-mokosh-private-${USER}`, but the compose.dev-sso.yml overlay marked it external with the non-suffixed name `dev-mokosh-private`, and `ensure-private-network` created that same non-suffixed network. When the base `just dev` stack (deps on `dev-mokosh-private-${USER}`) and the `just dev-sso` overlay got mixed, the server landed on the empty `dev-mokosh-private` network while Postgres and the other deps sat on `dev-mokosh-private-${USER}`. The server could then no longer resolve the `postgres` hostname and crash-looped on the DB connect, leaving Traefik with no backend so `https://<user>-mokosh-api.a8n.run` returned its default "404 page not found".

Point the overlay network `name:` and the `ensure-private-network` recipe at the same per-developer name `dev-mokosh-private-${USER}` so every service in the layered stack shares one network. Refresh the now-stale comments in both files that still described the network as non-suffixed and shared.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

fix(dev): drop server's depends_on infisical so compose project is valid

Create release / Create release from merged PR (pull_request) Has been skipped

Details

Check / fmt + clippy + compile + tests (pull_request) Successful in 49s

Details

dd34ae9ab8

A recent merge moved Infisical behind the opt-in `infisical` compose profile, but left `server.depends_on.infisical` in place. A hard depends_on a profiled-out (and therefore "undefined") service makes Compose reject the entire project, so `just dev`, `just dev-sso`, and even `just down` all fail with `service "server" depends on undefined service "infisical": invalid compose project`.

The mokosh-server binary never reads Infisical at startup (it has zero Infisical references in main.rs; only the one-shot mokosh-bootstrap CLI talks to Infisical), so the dependency was already wrong. Remove it. Infisical stays available on demand via `just dev-infisical` / the `infisical` profile.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>