Skip to content

AI endpoint auth contract + production hardening (ai-001)#36

Open
jaeyunha wants to merge 1 commit into
stagingfrom
qa/ai-001-final
Open

AI endpoint auth contract + production hardening (ai-001)#36
jaeyunha wants to merge 1 commit into
stagingfrom
qa/ai-001-final

Conversation

@jaeyunha

Copy link
Copy Markdown
Member

AI endpoint auth contract plus production-readiness hardening (health liveness/readiness split, HTTPS config validation, prod Dockerfiles/IaC).


Preserved from QA-worktree cleanup on 2026-06-16. Web QA: biome lint clean; vitest green where unit tests exist. Rust build/test not run locally (would regenerate large target dirs). Opened against staging for review.

🤖 Generated with Claude Code

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 3222c996e3

ℹ️ 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".

) -> Result<Json<serde_json::Value>, (StatusCode, Json<ErrorEnvelope>)> {
let pool = state.db.as_ref().ok_or_else(database_unavailable)?;
let actor = AuthenticatedUser::optional_from_headers(&state, &headers).await?;
let actor = AuthenticatedUser::from_headers(&state, &headers).await?.0;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Update AI docs for required authentication

This switches the repository AI endpoint from optional to mandatory auth, and the same pattern is repeated for the PR and changelog handlers below, but the user-facing API catalog still says auth is “Optional signed opengithub session cookie” for all three AI endpoints (web/src/lib/api-docs.ts:96, web/src/lib/api-docs.ts:120, web/src/lib/api-docs.ts:139). Clients following /docs/api for public repositories will now make unauthenticated calls and receive 401 not_authenticated, so the docs/examples should be updated to describe the required session/PAT contract.

Useful? React with 👍 / 👎.

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.

1 participant