eric/mcp-obsidian
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: correctly parse legacy OBSIDIAN_HOST format 'host:port' to avoid double port in URL

Browse Source
This commit is contained in:
Eric X. Liu
2025-08-16 19:04:03 +00:00
parent 00245ef40b
commit 897822ecaa
1 changed files with 13 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
src/mcp_obsidian/obsidian.py
View File

@@ -84,10 +84,19 @@ class Obsidian():
host = parsed.hostname or '127.0.0.1' host = parsed.hostname or '127.0.0.1'
port = parsed.port or (27124 if protocol == 'https' else 27123) port = parsed.port or (27124 if protocol == 'https' else 27123)
else: else:
# Legacy hostname/IP only format # Support legacy formats
protocol = 'https' # 1) hostname/IP only
host = host_config # 2) hostname:port (no protocol)
port = 27124 if ':' in host_config:
# Treat as host:port and default protocol to https
parsed = urlparse(f'https://{host_config}')
protocol = 'https'
host = parsed.hostname or '127.0.0.1'
port = parsed.port or 27124
else:
protocol = 'https'
host = host_config
port = 27124
return protocol, host, port return protocol, host, port