Skip to content

fix(oauth): point proxy landing page to OAuth docs#61015

Merged
rafaeelaudibert merged 1 commit into
masterfrom
posthog-code/oauth-proxy-docs-link
Jun 1, 2026
Merged

fix(oauth): point proxy landing page to OAuth docs#61015
rafaeelaudibert merged 1 commit into
masterfrom
posthog-code/oauth-proxy-docs-link

Conversation

@rafaeelaudibert
Copy link
Copy Markdown
Member

Problem

The OAuth proxy's root route returns a plain-text landing message, but it linked to the MCP docs (https://posthog.com/docs/model-context-protocol). This is the OAuth proxy, so the link should point to PostHog's OAuth integration docs. The message also used an em-dash, which rendered as mojibake under the text/plain response.

Changes

  • Point the landing page link to https://posthog.com/docs/api/oauth.
  • Replace the em-dash with a plain hyphen so the text renders cleanly.

How did you test this code?

I'm an agent. The existing tests/router.test.ts assertions (content-type contains text/plain, body contains PostHog OAuth Proxy) remain satisfied by inspection. I was unable to execute the vitest suite in this environment because the service's node_modules are not installed here.

Automatic notifications

  • Publish to changelog?
  • Alert Sales and Marketing teams?

Docs update

🤖 Agent context

Authored with Claude Code. The task was to repoint the OAuth proxy landing route from MCP docs to OAuth docs and fix the em-dash. I located the correct OAuth docs URL (/docs/api/oauth) via the PostHog docs search, then switched the em-dash to a plain hyphen at the user's request. Left the genuinely MCP-related references in services/mcp/ and the OAuth metadata service_documentation field untouched, since those are not the landing route in question.

The OAuth proxy root route advertised the MCP docs URL. Point it to the OAuth integration docs (https://posthog.com/docs/api/oauth) instead, and use a plain hyphen in the landing text to avoid the em-dash rendering as mojibake under text/plain.

Generated-By: PostHog Code
Task-Id: 107cea25-6ee2-42b0-b844-2b26e58db63a
@rafaeelaudibert rafaeelaudibert requested review from a team, MattBro, Piccirello and fercgomes and removed request for a team June 1, 2026 20:33
@graphite-app graphite-app Bot added the stamphog Request AI review from stamphog label Jun 1, 2026
@rafaeelaudibert rafaeelaudibert requested a review from a team June 1, 2026 20:33
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Jun 1, 2026

Reviews (1): Last reviewed commit: "fix(oauth): point proxy landing page to ..." | Re-trigger Greptile

@rafaeelaudibert rafaeelaudibert enabled auto-merge (squash) June 1, 2026 20:34
Copy link
Copy Markdown

@stamphog stamphog Bot left a comment

Choose a reason for hiding this comment

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

The change is a trivial URL update in the OAuth proxy landing page response, but it touches auth infrastructure (deny-listed) and has zero reviews. Request a review from a team member familiar with the OAuth proxy before merging.

@stamphog stamphog Bot removed the stamphog Request AI review from stamphog label Jun 1, 2026
@rafaeelaudibert rafaeelaudibert merged commit 5efb111 into master Jun 1, 2026
187 of 222 checks passed
@rafaeelaudibert rafaeelaudibert deleted the posthog-code/oauth-proxy-docs-link branch June 1, 2026 20:41
@deployment-status-posthog
Copy link
Copy Markdown

deployment-status-posthog Bot commented Jun 1, 2026

Deploy status

Environment Status Deployed At Workflow
dev ✅ Deployed 2026-06-01 21:12 UTC Run
prod-us ✅ Deployed 2026-06-01 21:29 UTC Run
prod-eu ✅ Deployed 2026-06-01 21:33 UTC Run

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants