Skip to content

Docs: clarify server URL paths for ChatGPT integration #1017

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

Merged
merged 2 commits into from
Jul 5, 2025

Conversation

thap2331
Copy link
Contributor

@thap2331 thap2331 commented Jul 2, 2025

What does this PR do?

  • Updates the docs/integrations/chatgpt.mdx file to clarify that users need to manually append the correct path to their server URL.

Why?

  • Prevent confusion and ensure users know they must include the correct path in the URL for proper functionality.

Notes

  • No functional code changes, just documentation.

@github-actions github-actions bot added the documentation Improvements or additions to documentation label Jul 2, 2025
Copy link
Owner

@jlowin jlowin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks but this is not necessarily correct - /sse/ is the default URL for SSE transport only, but users could change it (and it is the wrong url for streamable http, which is what this example doc uses). I think it would be better to say something like "the full URL to your MCP server" and mention including the path which defaults to /sse/ for SSE transport and /mcp/ for MCP transport

@thap2331
Copy link
Contributor Author

thap2331 commented Jul 3, 2025

Thanks but this is not necessarily correct - /sse/ is the default URL for SSE transport only, but users could change it (and it is the wrong url for streamable http, which is what this example doc uses). I think it would be better to say something like "the full URL to your MCP server" and mention including the path which defaults to /sse/ for SSE transport and /mcp/ for MCP transport

Thanks for catching that! I realized the error a bit later and was planning to update it—sorry for the inconvenience. Also, I just updated the doc.

@thap2331 thap2331 changed the title Docs: add /sse/ URL note for chatgpt Docs: add proper path to URL for chatgpt integration Jul 3, 2025
@thap2331 thap2331 changed the title Docs: add proper path to URL for chatgpt integration Docs: clarify server URL paths for ChatGPT integration Jul 3, 2025
Copy link
Owner

@jlowin jlowin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@jlowin jlowin merged commit dba9f2c into jlowin:main Jul 5, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants