didericis/bot-bottle
1
0
Fork 0
Code Issues 7 Pull Requests 15 Actions Packages Projects Releases Wiki Activity

Stop injecting Codex API-key placeholder #108

Merged
didericis merged 2 commits from codex/codex-device-auth-runtime into main 2026-05-29 02:48:16 -04:00
Conversation 2 Commits 2 Files Changed 9
+85 -41
didericis-codex commented 2026-05-29 02:40:14 -04:00
Collaborator
Copy Link
Copy Source

Summary

  • stop forwarding the non-secret OPENAI_API_KEY placeholder for Codex bottles
  • keep codex_auth accepted only as a legacy route marker
  • document device-login egress for auth.openai.com and add provider runtime tests

Tests

  • python3 -m unittest discover tests/unit
## Summary - stop forwarding the non-secret OPENAI_API_KEY placeholder for Codex bottles - keep codex_auth accepted only as a legacy route marker - document device-login egress for auth.openai.com and add provider runtime tests ## Tests - python3 -m unittest discover tests/unit
didericis-codex added 1 commit 2026-05-29 02:40:15 -04:00
fix(codex): stop injecting api key placeholder
test / unit (pull_request) Successful in 27s
Details
test / integration (pull_request) Successful in 41s
Details
cea832b21d
didericis reviewed 2026-05-29 02:42:22 -04:00
bot_bottle/manifest.py Outdated
@@ -188,6 +188,11 @@ EGRESS_AUTH_SCHEMES = ("Bearer", "token")
# logic — declare the role on whichever route
# injects the OAuth header.
#
# codex_auth: legacy marker for Codex API-key-style egress routes.
didericis commented 2026-05-29 02:42:22 -04:00
Owner
Copy Link
Copy Source

call this a placeholder marker rather than a legacy marker, and cite the reason for keeping it for follow up updates for better credential injection behavior

call this a placeholder marker rather than a legacy marker, and cite the reason for keeping it for follow up updates for better credential injection behavior
didericis-codex marked this conversation as resolved
didericis added 1 commit 2026-05-29 02:45:21 -04:00
docs(codex): clarify codex auth marker
test / unit (pull_request) Successful in 28s
Details
test / integration (pull_request) Successful in 40s
Details
test / unit (push) Successful in 28s
Details
test / integration (push) Successful in 42s
Details
dcaee53cec
didericis approved these changes 2026-05-29 02:48:01 -04:00
didericis merged commit dcaee53cec into main 2026-05-29 02:48:16 -04:00
didericis deleted branch codex/codex-device-auth-runtime 2026-05-29 02:48:16 -04:00
Sign in to join this conversation.
Reviewers
No Reviewers
didericis
Labels
Clear labels
Compat/Breaking
Breaking change that won't be backward compatible
 Kind/Bug
Something is not working
 Kind/Documentation
Documentation changes
 Kind/Enhancement
Improve existing functionality
 Kind/Feature
New functionality
 Kind/Security
This is security issue
 Kind/Testing
Issue or pull request related to testing
Priority
Critical
1
The priority is critical
Priority
High
2
The priority is high
Priority
Low
4
The priority is low
Priority
Medium
3
The priority is medium
Reviewed
Confirmed
1
Issue has been confirmed
Reviewed
Duplicate
2
This issue or pull request already exists
Reviewed
Invalid
3
Invalid issue
Reviewed
Won't Fix
3
This issue won't be fixed
Status
Abandoned
3
Somebody has started to work on this but abandoned work
Status
Blocked
1
Something is blocking this issue or pull request
Status
Need More Info
2
Feedback is required to reproduce issue or to continue work
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
didericis-claude (didericis (claude)) didericis-codex (didericis (codex)) didericis-gemma (gemma) didericis
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: didericis/bot-bottle#108
Delete Branch "codex/codex-device-auth-runtime"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?