Repository navigation
docs: add ChatGPT Apps SDK readiness guidance - #177
Conversation
|
Claude finished @mashkovd's task in 53s —— View job Code Review
Both changed files ( |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 6297c6cea6
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| - Privacy: `https://<your-host>/privacy` | ||
| 4. Verify OAuth discovery: | ||
| - `https://<your-host>/.well-known/oauth-protected-resource` | ||
| - `https://<your-host>/.well-known/oauth-authorization-server` |
There was a problem hiding this comment.
Resolve OAuth metadata URL from advertised auth server
Update this instruction to avoid hardcoding the authorization-server metadata on <your-host>. In shared-hmac / shared-hmac-legacy mode, the app intentionally does not mount /.well-known/oauth-authorization-server on the local host (cmd/server/main.go around the OAuth route wiring), and selectAuthServer points discovery to https://api.mctl.ai instead (cmd/server/main.go, selectAuthServer). As written, operators following this step can get a false failure (404 on <your-host>) even when OAuth discovery is correctly configured.
Useful? React with 👍 / 👎.
Summary
/docscovering public pages, OAuth discovery, MCP protocol health, safety defaults, user controls, and submission package prepValidation
git diff --checkgo test ./internal/web ./cmd/server/,/docs,/security,/privacy,/.well-known/oauth-protected-resource, and/.well-known/oauth-authorization-server/mcpinitialize probe returned HTTP 401 withauthentication required, confirming the protected endpoint is reachableget_my_audit_log(limit=1)succeeded through the configured mctl-telegram connectorReferences