From 357482cfda73501bb4dcfbbffc46244476dc5a65 Mon Sep 17 00:00:00 2001
From: Janik von Rotz <login@janikvonrotz.ch>
Date: Mon, 28 Jul 2025 15:56:04 +0200
Subject: [PATCH] fix(odoo-mcp): update ODOO_URL environment variable to use
 service name instead of localhost

---
 images/odoo-mcp/Dockerfile | 11 +----------
 images/odoo-mcp/README.md  |  2 +-
 2 files changed, 2 insertions(+), 11 deletions(-)

diff --git a/images/odoo-mcp/Dockerfile b/images/odoo-mcp/Dockerfile
index e9aa8e47..7ebe06ab 100644
--- a/images/odoo-mcp/Dockerfile
+++ b/images/odoo-mcp/Dockerfile
@@ -1,29 +1,20 @@
 FROM python:3.11-slim
 
-# Set working directory
 WORKDIR /app
 
-# Install system dependencies
 RUN apt-get update && apt-get install -y \
     gcc \
     && rm -rf /var/lib/apt/lists/*
 
-# Install Python dependencies
 RUN pip install --no-cache-dir fastmcp
 
-# Copy the server script
 COPY ./images/odoo-mcp/bin/server /app/server
 
-# Make the server script executable
 RUN chmod +x /app/server
 
-# Set environment variables with defaults
-ENV ODOO_URL=http://localhost:8069
+ENV ODOO_URL=http://odoo:8069
 ENV ODOO_DATABASE=odoo
 ENV ODOO_USERNAME=admin
 ENV ODOO_PASSWORD=admin
 
-# Expose any necessary ports (MCP typically uses stdio, so this may not be needed)
-
-# Run the server
 CMD ["python3", "/app/server"]
diff --git a/images/odoo-mcp/README.md b/images/odoo-mcp/README.md
index f35d2878..730460ed 100644
--- a/images/odoo-mcp/README.md
+++ b/images/odoo-mcp/README.md
@@ -30,7 +30,7 @@ Add this config to your MCP client configuration:
                 "mintsystem/odoo-mcp"
             ],
             "env": {
-                "ODOO_URL": "http://localhost:8069",
+                "ODOO_URL": "http://odoo:8069",
                 "ODOO_DATABASE": "18.0",
                 "ODOO_USERNAME": "admin",
                 "ODOO_PASSWORD": "admin"