Skip to content

feat: configurable timezone for docker in time server #639

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

Open
S1M0N38 opened this issue Feb 19, 2025 · 0 comments · May be fixed by #640
Open

feat: configurable timezone for docker in time server #639

S1M0N38 opened this issue Feb 19, 2025 · 0 comments · May be fixed by #640
Labels
enhancement New feature or request server-time Reference implementation for the Time MCP server - src/time

Comments

@S1M0N38
Copy link

S1M0N38 commented Feb 19, 2025

I'd like to set the timezone in the src/servers/time server when using a docker installation. Right the entry point of the docker file does not include the available args:

ENTRYPOINT ["mcp-server-time"]

I'd like to be able to set --local-timezone using a env variable. That is

"mcpServers": {
  "time": {
    "command": "docker",
    "args": [
      "run",
      "-i",
      "--rm",
      "-e",
      "LOCAL_TIMEZONE",
      "mcp/time"
    ],
    "env": {
      "LOCAL_TIMEZONE": "America/New_York"
    }
  }
}
@S1M0N38 S1M0N38 added the enhancement New feature or request label Feb 19, 2025
S1M0N38 added a commit to S1M0N38/servers that referenced this issue Feb 19, 2025
Fixes modelcontextprotocol#639

Add support for configurable timezone in Docker for `time` server.

* Update `src/time/Dockerfile` to include `LOCAL_TIMEZONE` environment variable.
* Set default timezone to "UTC" in `src/time/Dockerfile`.
* Modify `ENTRYPOINT` in `src/time/Dockerfile` to include `--local-timezone` argument.
* Update `src/time/README.md` to include instructions on setting the timezone using `LOCAL_TIMEZONE` environment variable.

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/modelcontextprotocol/servers/issues/639?shareId=XXXX-XXXX-XXXX-XXXX).
@S1M0N38 S1M0N38 linked a pull request Feb 19, 2025 that will close this issue
@olaservo olaservo added the server-time Reference implementation for the Time MCP server - src/time label May 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request server-time Reference implementation for the Time MCP server - src/time
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants