github/AutoGPT
1
1
Fork 0
mirror of https://github.com/Significant-Gravitas/AutoGPT.git synced 2026-02-13 00:05:02 -05:00
Code Issues Packages Projects Releases Wiki Activity

fix(backend/mcp): Add defensive null check for credential metadata access

Browse Source 
Guard against NoneType AttributeError when cred.metadata is None by
using (cred.metadata or {}).get() pattern consistently.
This commit is contained in:
Zamil Majdy
2026-02-12 19:38:00 +04:00
parent 8663d7a5ba
commit c8de1d6dd9
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
autogpt_platform/backend/backend/api/features/mcp/routes.py
View File

@@ -86,7 +86,7 @@ async def discover_tools(
for cred in mcp_creds: for cred in mcp_creds:
if ( if (
isinstance(cred, OAuth2Credentials) isinstance(cred, OAuth2Credentials)
and cred.metadata.get("mcp_server_url") == request.server_url and (cred.metadata or {}).get("mcp_server_url") == request.server_url
): ):
if best_cred is None or ( if best_cred is None or (
(cred.access_token_expires_at or 0) (cred.access_token_expires_at or 0)
@@ -353,7 +353,7 @@ async def mcp_oauth_callback(
for old in old_creds: for old in old_creds:
if ( if (
isinstance(old, OAuth2Credentials) isinstance(old, OAuth2Credentials)
and old.metadata.get("mcp_server_url") == meta["server_url"] and (old.metadata or {}).get("mcp_server_url") == meta["server_url"]
): ):
await creds_manager.store.delete_creds_by_id(user_id, old.id) await creds_manager.store.delete_creds_by_id(user_id, old.id)
logger.info( logger.info(