Skip to content

Commit 0a2fe11

Browse files
author
Greg Bowler
authored
Merge pull request #64 from TheLastCicada/master
[Security Issue] Update README.md to recommend removing auth.json after use
2 parents d936bcb + 4d838ad commit 0a2fe11

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

README.md

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -208,7 +208,7 @@ It's recommended to use SSH keys for authentication, but sometimes HTTP basic au
208208
}
209209
```
210210

211-
3) Pass this secret to auth.json as a separate action step within your Yaml config:
211+
3) Pass this secret to auth.json as a separate action step within your Yaml config, and remove auth.json to prevent deploying it:
212212

213213
```yaml
214214
jobs:
@@ -221,6 +221,9 @@ jobs:
221221
222222
- name: Install dependencies
223223
uses: php-actions/composer@v6
224+
225+
- name: Remove auth.json file
226+
run: rm -f $GITHUB_WORKSPACE/auth.json
224227
```
225228

226229
4) Now, any connections Composer makes to github.com will use your HTTP basic auth credentials, which is essentially the same as being logged in as you, so your private repositories will now be available to Composer.

0 commit comments

Comments
 (0)