Skip to content

fix: Local API requires docker exec when running with Docker Compose#285

Closed
rud3ka wants to merge 1 commit intodocumentationfrom
fix/create-super-administrator-docker-instructions
Closed

fix: Local API requires docker exec when running with Docker Compose#285
rud3ka wants to merge 1 commit intodocumentationfrom
fix/create-super-administrator-docker-instructions

Conversation

@rud3ka
Copy link
Copy Markdown
Collaborator

@rud3ka rud3ka commented Apr 28, 2026

Summary

Fixes incorrect instructions in create-super-administrator.md found during hands-on testing of the Docker Compose setup.

  • Add warning: Local API is only accessible from within the Core container when running with Docker Compose — calling localhost:8080 directly from the host machine returns HTTP 401
  • Add docker exec example: correct way to call Local API in a Docker Compose setup
  • Fix URL scheme: https://localhosthttp://localhost (Local API does not use TLS)
  • Clarify context: label existing examples as "standalone deployment" to distinguish from Docker Compose

Test plan

  • Follow create-super-administrator.md with a Docker Compose setup
  • Verify that docker exec core curl ... successfully creates the admin
  • Verify that calling localhost:8080 directly from host returns 401

🤖 Generated with Claude Code

…cker Compose

- Add warning that Local API returns 401 when called from host machine
- Add docker exec example for Docker Compose deployments
- Clarify standalone vs Docker Compose usage for both curl and wget examples
- Fix curl example: https -> http (Local API does not use TLS)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Apr 28, 2026

CLA assistant check
All committers have signed the CLA.

@3keyroman
Copy link
Copy Markdown
Contributor

Hi @rud3ka , we do not support deployment using docker compose. This is rather description for the developer, when they are working with local environment. We support currently only Helm chart and virtual appliance.

@rud3ka
Copy link
Copy Markdown
Collaborator Author

rud3ka commented Apr 28, 2026

Thank you for the clarification! You're right — the docker exec note is specific to the Docker Compose developer workflow and doesn't belong in the general installation guide. I'll close this PR. The fix has been applied where it's actually needed: in the development-environment README (PR #13) and in the QA local setup guide (PR #286).

@rud3ka rud3ka closed this Apr 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants