Skip to content

Support STDIO transport#9

Merged
berggren merged 4 commits intomainfrom
stdio
Aug 31, 2025
Merged

Support STDIO transport#9
berggren merged 4 commits intomainfrom
stdio

Conversation

@berggren
Copy link
Copy Markdown
Contributor

Summary

Updates the OpenRelik MCP server to use environment variables for configuration, fixes a bug where host and port where not correctly passed and improves API client instantiation.

Technical Details:

  • Add STDIO transport
  • Docker Compose Configuration: Introduced environment variables for MCP transport protocol, host, and port to allow configuration via docker-compose.yml.
  • Main Script Updates:
    • Added deprecation warning for the sse transport.
    • Passed host and port to mcp.run when using http or sse transport
  • License Headers: Added license headers to src/openrelik_mcp_server/tools.py and src/openrelik_mcp_server/utils.py.

@berggren berggren requested a review from hacktobeer August 30, 2025 23:58
Copy link
Copy Markdown
Contributor

@hacktobeer hacktobeer left a comment

Choose a reason for hiding this comment

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

Small nit to update README as well with new envs.

environment:
- OPENRELIK_API_URL=${OPENRELIK_API_URL}
- OPENRELIK_API_KEY=${OPENRELIK_API_KEY}
- MCP_TRANSPORT=http
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can you update the README as well?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can you add the environment variabled to the README as well?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good catch, fixing

@berggren berggren merged commit 3d6c294 into main Aug 31, 2025
2 checks passed
@berggren berggren deleted the stdio branch August 31, 2025 09:54
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