Skip to content

Commit 93ec867

Browse files
authored
Merge pull request #1728 from abhayclasher/security-hardening-workflow
Security: Harden codex-review workflow against script injection
2 parents d6f66e9 + 48df4ae commit 93ec867

3 files changed

Lines changed: 15 additions & 2 deletions

File tree

.github/workflows/codex-review.yml

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,10 @@ jobs:
3636
- name: Run Codex review
3737
id: run_codex
3838
uses: openai/codex-action@v1
39+
env:
40+
# Use env variables to handle untrusted metadata safely
41+
PR_TITLE: ${{ github.event.pull_request.title }}
42+
PR_BODY: ${{ github.event.pull_request.body }}
3943
with:
4044
openai-api-key: ${{ secrets.OPENAI_API_KEY }}
4145
prompt: |
@@ -53,8 +57,8 @@ jobs:
5357
5458
Pull request title and body:
5559
----
56-
${{ github.event.pull_request.title }}
57-
${{ github.event.pull_request.body }}
60+
$PR_TITLE
61+
$PR_BODY
5862
5963
post-feedback:
6064
runs-on: ubuntu-latest

changelog.md

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,11 @@
1+
1.65.2 (2026/03/19)
2+
==============
3+
4+
Security
5+
--------
6+
* Harden `codex-review` workflow against script injection from untrusted PR metadata.
7+
8+
19
1.65.1 (2026/03/18)
210
==============
311

mycli/AUTHORS

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -113,6 +113,7 @@ Contributors:
113113
* tmijieux
114114
* Scott Nemes
115115
* Angelino Storm
116+
* Abhay Kumar
116117

117118

118119
Created by:

0 commit comments

Comments
 (0)