Skip to content

Authenticate API GitHub reads with the Integrations App - #316

Merged
admdly merged 9 commits into
mainfrom
codex/api-github-app-auth
Nov 13, 2026
Merged

admdly merged 9 commits into
mainfrom
codex/api-github-app-auth

Conversation

@admdly

@admdly admdly commented Nov 12, 2026 •

Copy link
Copy Markdown
Contributor

Replace the API's long-lived GitHub PAT with short-lived FOSSBilling Integrations installation tokens, restricted to FOSSBilling/FOSSBilling and Contents, Actions and Pull requests read permissions.

A shared lazy Octokit REST/GraphQL client serves versions, stats, previews and developer verification. @octokit/auth-app manages token issuance, caching and refresh. Cache hits, preview budgets and no-op updates keep their existing behavior; authentication outages remain unavailable rather than becoming missing users or anonymous requests.

@github-actions

github-actions Bot commented Nov 12, 2026 •

Copy link
Copy Markdown

Deploying Preview to Cloudflare (api)

Preview URL: https://codex-api-github-app-auth-api.fossbilling.workers.dev/ (commit 011dadd)

This URL reflects your latest successful Preview deployment.

Preview Deployments by commit

Status Deployment URL Commit Updated (UTC) Details
Build: Success ✅
Deployment: Success ✅
https://af9df079-api.fossbilling.workers.dev/ 011dadd 2026-11-13T00:17:42.017Z View logs ↗
Build: Success ✅
Deployment: Success ✅
https://a8ad0fe6-api.fossbilling.workers.dev/ 9a35ff1 2026-11-12T23:55:03.081Z View logs ↗
Build: Success ✅
Deployment: Success ✅
https://58292592-api.fossbilling.workers.dev/ f68a68c 2026-11-12T23:21:26.632Z View logs ↗
Build: Success ✅
Deployment: Success ✅
https://09e5aadb-api.fossbilling.workers.dev/ 5f21050 2026-11-12T23:00:21.010Z View logs ↗
Build: Success ✅
Deployment: Success ✅
https://a4b0e283-api.fossbilling.workers.dev/ 27ce96d 2026-11-12T21:43:29.438Z View logs ↗
Build: Success ✅
Deployment: Success ✅
https://b0d57f28-api.fossbilling.workers.dev/ 1fb3386 2026-11-12T20:50:29.442Z View logs ↗
Build: Success ✅
Deployment: Success ✅
https://d5f8eb0b-api.fossbilling.workers.dev/ dc70a28 2026-11-12T20:27:37.999Z View logs ↗

Ignore all environment and dev vars files while allowing example dev vars files. Update the private key format and clarify the shared assertion secret.
@admdly
admdly marked this pull request as ready for review November 12, 2026 23:02
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 12, 2026 •

Copy link
Copy Markdown
Codex Review Summary: ✅ Completed

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
🔒 Security Review ✅ Completed 2026-11-12T23:06:06.638285Z 5f21050 Draft marked ready
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@admdly admdly self-assigned this Nov 12, 2026

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All reported issues were addressed across 57 files

Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic

Comment thread services/api/package.json Resolved
Comment thread services/api/README.md Resolved

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

0 issues found across 5 files (changes from recent commits).

Requires human review: Migrates API GitHub reads from a long-lived PAT to short-lived GitHub App installation tokens, adding new GITHUB_APP_* secrets and build/deploy configuration. Because this changes production authentication and credential handling, it requires human sign-off.

Turn on auto-fix | Re-trigger cubic

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

0 issues found across 2 files (changes from recent commits).

Requires human review: Migrates API GitHub reads from a long-lived PAT to GitHub App installation-token auth, adding GITHUB_APP_* secrets and build/deploy config. Decisive: this changes production authentication and credential handling, so a human must approve App scope and confirm the appId/Client ID wiring.

Turn on auto-fix | Re-trigger cubic

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

0 issues found across 1 file (changes from recent commits).

Requires human review: Migrates API GitHub reads from a long-lived PAT to GitHub App installation-token authentication, adding GITHUB_APP_* secrets, build conditions, and a shared Octokit client. Decisive: production authentication and credential handling change, and the App ID/Client ID wiring still needs human sign-off.

Turn on auto-fix | Re-trigger cubic

@admdly
admdly merged commit ccfb3d7 into main Nov 13, 2026
23 checks passed
@admdly
admdly deleted the codex/api-github-app-auth branch November 13, 2026 00:18
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