Skip to content

ci: fix CodeQL permissions for private repositories - #3606

Merged
apcha-oai merged 1 commit into
mainfrom
apcha/fix-codeql-private-repo-permissions
Aug 13, 2026
Merged

ci: fix CodeQL permissions for private repositories#3606
apcha-oai merged 1 commit into
mainfrom
apcha/fix-codeql-private-repo-permissions

Conversation

@apcha-oai

Copy link
Copy Markdown
Contributor

Summary

  • Grant the CodeQL workflow actions: read alongside its existing contents: read and security-events: write permissions.
  • Allow the shared SDK workflow to read workflow-run metadata when it runs in the mirrored private repository.

Why

The public workflow currently succeeds because public Actions workflow-run metadata can be read without an explicit Actions permission. The identical workflow fails in the private mirror with Resource not accessible by integration while preparing to upload CodeQL results.

Validation

@apcha-oai
apcha-oai requested a review from a team as a code owner August 13, 2026 05:46
@openai-sdks

openai-sdks Bot commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

OkTest Summary

237/237 SDK tests passed in 11.391s for Python SDK PR #3606.

Test results — 42 files
Test Result Time
tests/chat-completions-complex-body.test.ts ✅ Passed 183ms
tests/chat-completions-create.test.ts ✅ Passed 261ms
tests/chat-completions-stream.test.ts ✅ Passed 133ms
tests/files-content-binary.test.ts ✅ Passed 245ms
tests/files-create-multipart.test.ts ✅ Passed 181ms
tests/files-list-pagination.test.ts ✅ Passed 166ms
tests/initialize-config.test.ts ✅ Passed 200ms
tests/instance-isolation.test.ts ✅ Passed 216ms
tests/models-list.test.ts ✅ Passed 284ms
tests/responses-background-lifecycle.test.ts ✅ Passed 269ms
tests/responses-body-method-errors.test.ts ✅ Passed 296ms
tests/responses-cancel-timeout.test.ts ✅ Passed 209ms
tests/responses-cancel.test.ts ✅ Passed 193ms
tests/responses-compact-retries.test.ts ✅ Passed 329ms
tests/responses-compact.test.ts ✅ Passed 286ms
tests/responses-create-advanced-stream.test.ts ✅ Passed 324ms
tests/responses-create-advanced.test.ts ✅ Passed 252ms
tests/responses-create-disconnect.test.ts ✅ Passed 1.203s
tests/responses-create-errors.test.ts ✅ Passed 181ms
tests/responses-create-malformed-api-responses.test.ts ✅ Passed 248ms
tests/responses-create-retries.test.ts ✅ Passed 282ms
tests/responses-create-stream-failures.test.ts ✅ Passed 134ms
tests/responses-create-stream-timeout.test.ts ✅ Passed 230ms
tests/responses-create-stream-wire.test.ts ✅ Passed 3.793s
tests/responses-create-stream.test.ts ✅ Passed 98ms
tests/responses-create-terminal-states.test.ts ✅ Passed 254ms
tests/responses-create-timeout.test.ts ✅ Passed 214ms
tests/responses-create.test.ts ✅ Passed 279ms
tests/responses-delete.test.ts ✅ Passed 226ms
tests/responses-input-items-errors.test.ts ✅ Passed 329ms
tests/responses-input-items-list.test.ts ✅ Passed 361ms
tests/responses-input-items-options.test.ts ✅ Passed 272ms
tests/responses-input-tokens-count-timeout.test.ts ✅ Passed 253ms
tests/responses-input-tokens-count.test.ts ✅ Passed 235ms
tests/responses-malformed-inputs.test.ts ✅ Passed 2.887s
tests/responses-not-found-errors.test.ts ✅ Passed 252ms
tests/responses-parse.test.ts ✅ Passed 413ms
tests/responses-retrieve-retries.test.ts ✅ Passed 226ms
tests/responses-retrieve.test.ts ✅ Passed 280ms
tests/responses-stored-method-errors.test.ts ✅ Passed 626ms
tests/retry-behavior.test.ts ✅ Passed 3.201s
tests/sdk-error-shape.test.ts ✅ Passed 297ms

View OkTest run #31671505234

SDK merge (e4fcae6dbf73) · head (bc7a5786fc84) · base (a1eeab58db02) · OkTest (a845fa206fa4)

@apcha-oai
apcha-oai enabled auto-merge August 13, 2026 06:06
@apcha-oai
apcha-oai added this pull request to the merge queue Aug 13, 2026
Merged via the queue into main with commit 5578d39 Aug 13, 2026
20 checks passed
@apcha-oai
apcha-oai deleted the apcha/fix-codeql-private-repo-permissions branch August 13, 2026 15:56
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