Fix healthcheck to not use env variable

[yannis-srl]

Description

Changing the healthcheck in the example docker compose to not use the environment variable which only specifies the exposed port.

Motivation and Context

This PR addresses: #862

This makes it possible to change the ports by setting the env variables, without bloodhound breaking.

How Has This Been Tested?

I have run it to avoid syntax errors by mistyping.

Screenshots (optional):

Types of changes

• Bug fix (non-breaking change which fixes an issue)

Checklist:

• I have met the contributing prerequisites
  • Assigned myself to this PR (I do not have the permissions to do this. I have done all other steps.)
  • Added the appropriate labels
  • Associated an issue: Issues and Pull Requests README #672
  • Read the Contributing guide: https://github.com/SpecterOps/BloodHound/wiki/Contributing
• I have ensured that related documentation is up-to-date
  • Open API docs
  • Code comments (GoDocs / JSDocs)
• I have followed proper test practices
  • Added/updated tests to cover my changes
  • All new and existing tests passed

[github-actions]

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

[yannis-srl]

I have read the CLA Document and I hereby sign the CLA

[superlinkx]

I'm checking into this issue today. I actually think this looks correct, but running some scenarios to validate

[superlinkx]

Tested on bare metal Linux/Docker and Docker Desktop for Mac, both worked exactly as expected with these changes. Thanks for fixing my bug!

[superlinkx]

@yannis-srl if you wouldn't mind rebasing on the latest main branch, that should fix the vulnerability reporting failure and allow for merging

[StephenHinck]

@yannis-srl - Thank you very much for finding and fixing this issue! I would love to send you a swag package to show our gratitude for your contributions to the product. If you can email me at shinck (AT) specterops (DOT) io with your shipping address and t-shirt size, I'll get that on its way!