Skip to content

Commit d7d176d

Browse files
committed
fix: pin 1password CLI action and fix gh secret set stdin handling
Pin 1password/install-cli-action to commit SHA (v2.0.2) for supply chain security. Remove erroneous --body - flag from gh secret set which passed the literal string "-" instead of reading piped stdin. https://claude.ai/code/session_01SvzkZUEyQnbHgMWodBoq65
1 parent 64ffafd commit d7d176d

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

.github/actions/1password-secret-sync/action.yml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,8 @@ runs:
3636
using: 'composite'
3737
steps:
3838
- name: Install 1Password CLI
39-
uses: 1password/install-cli-action@v2
39+
# Pin to commit SHA for supply chain security — update when action changes
40+
uses: 1password/install-cli-action@9a0c9dd934086b7ab1d90115d455bda1c53c2bdb # v2.0.2
4041

4142
- name: Sync secrets
4243
id: sync
@@ -113,7 +114,7 @@ runs:
113114
echo "[DRY RUN] Would set '$target_name' on $target_repo"
114115
skipped=$((skipped + 1))
115116
else
116-
if echo "$value" | gh secret set "$target_name" --repo "$target_repo" --body -; then
117+
if echo "$value" | gh secret set "$target_name" --repo "$target_repo"; then
117118
echo "Set '$target_name' on $target_repo"
118119
synced=$((synced + 1))
119120
else

0 commit comments

Comments
 (0)