mike/open-project-sidecar-mcp
1
1
Fork 0
Code Issues Pull requests Releases 34 Packages 2 Activity Actions

Release v1.4.20-alpha #34

Merged
Mike Bros merged 1 commit from release/1.4.20-alpha into main 2026-02-16 08:13:35 +00:00
Conversation 1 Commits 1 Files changed 3 +117 -15
Gravity Bot commented 2026-02-16 08:02:51 +00:00
Contributor
Copy link

Summary

  • Fix Docker sidecar setup flow missing hostname prompt
  • dockerSetupFlow() now interactively prompts for OpenProject domain instead of silently auto-detecting
  • generateMcpServiceYaml() uses the resolved external URL as OPENPROJECT_URL so the Host header matches OpenProject's host_name config
  • 7 new tests for hostname extraction and YAML generation

Test plan

  • npm test — 238 tests passing
  • Manual: run setup wizard, choose Docker sidecar > new setup — should prompt for domain
  • Verify generated YAML contains external URL in OPENPROJECT_URL
## Summary - Fix Docker sidecar setup flow missing hostname prompt - `dockerSetupFlow()` now interactively prompts for OpenProject domain instead of silently auto-detecting - `generateMcpServiceYaml()` uses the resolved external URL as `OPENPROJECT_URL` so the Host header matches OpenProject's `host_name` config - 7 new tests for hostname extraction and YAML generation ## Test plan - [ ] `npm test` — 238 tests passing - [ ] Manual: run setup wizard, choose Docker sidecar > new setup — should prompt for domain - [ ] Verify generated YAML contains external URL in `OPENPROJECT_URL`
fix(setup): add hostname prompt to Docker sidecar setup flow
All checks were successful
CI / lint (pull_request) Successful in 10s
Details
CI / security (pull_request) Successful in 8s
Details
CI / test (pull_request) Successful in 17s
Details
f0c449c287 
The dockerSetupFlow() silently auto-detected OPENPROJECT_HOST__NAME from
the compose file and fell back to the internal Docker URL (http://openproject:80)
when not found. OpenProject rejects requests where the Host header doesn't
match its configured host_name, causing "Invalid host_name configuration" errors.

- Replace silent auto-detection with interactive hostname prompt
- Pre-fill from OPENPROJECT_HOST__NAME in compose if detected
- Probe HTTPS/HTTP via normalizeAndProbeUrl() with insecure fallback
- Pass resolved external URL to generateMcpServiceYaml() as OPENPROJECT_URL
- Add 7 tests for hostname extraction and YAML generation with external URL

Closes OP#605

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Mike Bros approved these changes 2026-02-16 08:13:28 +00:00
Mike Bros deleted branch release/1.4.20-alpha 2026-02-16 08:13:35 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Mike Bros
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
mike/open-project-sidecar-mcp!34
No description provided.