Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug: Healthcheck in example docker compose uses wrong port #862

Closed
3 tasks done
yannis-srl opened this issue Sep 17, 2024 · 2 comments · Fixed by #864
Closed
3 tasks done

Bug: Healthcheck in example docker compose uses wrong port #862

yannis-srl opened this issue Sep 17, 2024 · 2 comments · Fixed by #864
Labels
bug Something isn't working ticketed (automation only) Ticket has been created internally for tracking triage This issue requires triaging

Comments

@yannis-srl
Copy link
Contributor

Description:

When downloading the example docker compose from the link in the README and then changing the NEO4J_WEB_PORT environment variable the healthcheck breaks and the bloodhound ui will not start. This is because the env variable NEO4J_WEB_PORT is used in the healthcheck command, even tough it only affects the port exposed on the host not the port exposed inside the container.

Are you intending to fix this bug?

Yes.

Component(s) Affected:

  • Other (tooling, documentation, etc.)

Steps to Reproduce:

  1. Download https://github.com/SpecterOps/BloodHound/blob/main/examples/docker-compose/docker-compose.yml as specified in README.
  2. Set env variable NEO4J_WEB_PORT to any port, e.g. 16666
  3. Run docker compose pull && docker compose up

Expected Behavior:

All three containers start and the neo4j web interface is reachable via the specified port.

Actual Behavior:

The healthcheck fails, even tough the neo4j web interface is reachable and the bloodhound ui container never starts.

Environment Information:

BloodHound: v5.15

Collector: None

OS: MAC OS 14.6

Docker: 27.0.3

Additional Information:

As far as I can tell this is already fixed in the docker-compose.dev.yml and should just be a problem of an outdated example file.

Potential Solution (optional):

Hardcode the port of the healthcheck to 7474 (same for postgres with port 5432) because the internal port is also hardcoded.

Related Issues:

Contributor Checklist:

  • I have searched the issue tracker to ensure this bug hasn't been reported before or is not already being addressed.
  • I have provided clear steps to reproduce the issue.
  • I have included relevant environment information details.
@yannis-srl yannis-srl added bug Something isn't working triage This issue requires triaging labels Sep 17, 2024
@yannis-srl
Copy link
Contributor Author

#864 should solve this issue.

@slokie-so slokie-so added the ticketed (automation only) Ticket has been created internally for tracking label Oct 21, 2024
@slokie-so slokie-so linked a pull request Oct 21, 2024 that will close this issue
3 tasks
@StephenHinck
Copy link
Collaborator

This was closed with #864

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ticketed (automation only) Ticket has been created internally for tracking triage This issue requires triaging
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants