Skip to content

Added healthcheck endpoint#47

Merged
JoshBragg-Cohere merged 2 commits into
mainfrom
jbragg/healthcheck
Apr 16, 2026
Merged

Added healthcheck endpoint#47
JoshBragg-Cohere merged 2 commits into
mainfrom
jbragg/healthcheck

Conversation

@JoshBragg-Cohere
Copy link
Copy Markdown
Contributor

@JoshBragg-Cohere JoshBragg-Cohere commented Apr 14, 2026

When using NorthMCPServer directly a healthcheck endpoint at /health will be enabled by default. This is primarily targeted to be used by k8s to ensure MCP is healthy.

image

Note

Medium Risk
Adds a new unauthenticated /health route enabled by default, which slightly increases public surface area and could conflict with user-defined routes if they already use /health. The implementation is small but changes default server behavior.

Overview
NorthMCPServer now registers a built-in GET /health endpoint returning plain-text OK, enabled by default and bypassing auth; a new health_check=False constructor flag disables it.

Examples and docs are updated to rely on the built-in health check (removing the prior custom-route example), and tests are adjusted/added to assert /health works by default and returns 404 when disabled.

Reviewed by Cursor Bugbot for commit 937d382. Bugbot is set up for automated code reviews on this repo. Configure here.

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 8d46476. Configure here.

Comment thread src/north_mcp_python_sdk/__init__.py
@JoshBragg-Cohere JoshBragg-Cohere added this pull request to the merge queue Apr 16, 2026
Merged via the queue into main with commit 9e7be14 Apr 16, 2026
4 checks passed
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.

2 participants