[PR #161] [MERGED] fix: Enable environment variable inheritance for Smithery cloud services authentication #190

New issue

Closed

opened 2026-10-18 21:12:33 +00:00 by bsakaguchi · 0 comments

bsakaguchi commented

2026-10-18 21:12:33 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/open-webui/mcpo/pull/161
Author: @KyleF0X
Created: 5/28/2025
Status: ✅ Merged
Merged: 5/28/2025
Merged by: @tjbck

Base: dev ← Head: fix-smithery-authentication

📝 Commits (2)

27af7a6 Merge pull request #135 from taylorwilsdon/issues/73
438395c fix: Enable environment variable inheritance for Smithery cloud services authentication

📊 Changes

1 file changed (+1 additions, -1 deletions)

View changed files

📝 src/mcpo/main.py (+1 -1)

📄 Description

🔧 Fixes Issue with Smithery Cloud Services Authentication

Resolves: Authentication failures causing 500 Internal Server Errors when using Smithery cloud services through MCPO (related to Discussion #101)

🚨 Problem

MCPO was not passing environment variables to Smithery CLI subprocesses, causing authentication failures with cloud services like:

Context7 (library search & documentation)
Office Word (document automation)
Playwright (browser automation)

Error: {"detail": {"message": "Unexpected error", "error": ""}}

✅ Solution

Modified StdioServerParameters to inherit parent environment variables:

# Before
env={**env},

# After  
env={**os.environ, **env},

---

<sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

## 📋 Pull Request Information **Original PR:** https://github.com/open-webui/mcpo/pull/161 **Author:** [@KyleF0X](https://github.com/KyleF0X) **Created:** 5/28/2025 **Status:** ✅ Merged **Merged:** 5/28/2025 **Merged by:** [@tjbck](https://github.com/tjbck) **Base:** `dev` ← **Head:** `fix-smithery-authentication` --- ### 📝 Commits (2) - [`27af7a6`](https://github.com/open-webui/mcpo/commit/27af7a6f221944b92aacfc74c5f745753147dc72) Merge pull request #135 from taylorwilsdon/issues/73 - [`438395c`](https://github.com/open-webui/mcpo/commit/438395c768958e73412d8a39e2d1098e7217af9f) fix: Enable environment variable inheritance for Smithery cloud services authentication ### 📊 Changes **1 file changed** (+1 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `src/mcpo/main.py` (+1 -1) </details> ### 📄 Description ## 🔧 Fixes Issue with Smithery Cloud Services Authentication **Resolves:** Authentication failures causing 500 Internal Server Errors when using Smithery cloud services through MCPO (related to Discussion #101) ## 🚨 Problem MCPO was not passing environment variables to Smithery CLI subprocesses, causing authentication failures with cloud services like: - Context7 (library search & documentation) - Office Word (document automation) - Playwright (browser automation) **Error:** `{"detail": {"message": "Unexpected error", "error": ""}}` ## ✅ Solution Modified `StdioServerParameters` to inherit parent environment variables: ```python # Before env={**env}, # After env={**os.environ, **env}, --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

bsakaguchi

2026-10-18 21:12:33 +00:00