Skip to content

Change the default value of persist-credentials to false#1687

Open
michi-covalent wants to merge 1 commit into
actions:mainfrom
michi-covalent:pr/michi-covalent/persist-credentials
Open

Change the default value of persist-credentials to false#1687
michi-covalent wants to merge 1 commit into
actions:mainfrom
michi-covalent:pr/michi-covalent/persist-credentials

Conversation

@michi-covalent

Copy link
Copy Markdown

Change the default value of persist-credentials setting from true to false to reduce the risk of unintentionally exposing the GITHUB_TOKEN secret.

Fixes: #485

Change the default value of persist-credentials setting from true to
false to reduce the risk of unintentionally exposing the GITHUB_TOKEN
secret.

Fixes: actions#485

Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
@michi-covalent michi-covalent requested a review from a team as a code owner April 20, 2024 23:46
@michi-covalent

Copy link
Copy Markdown
Author

i'd like to get this shipped in the next major release.

vszakats added a commit to curl/curl that referenced this pull request Dec 16, 2024
Suggested by zizmor GHA analysis tool.

Also:
- Move GH variables within single-quotes.
- Prefer single-quotes in shell code. (tidy-up)

Ref: actions/checkout#485
Ref: actions/checkout#1687
Ref: https://woodruffw.github.io/zizmor/

Closes #15746
pps83 pushed a commit to pps83/curl that referenced this pull request Apr 26, 2025
Suggested by zizmor GHA analysis tool.

Also:
- Move GH variables within single-quotes.
- Prefer single-quotes in shell code. (tidy-up)

Ref: actions/checkout#485
Ref: actions/checkout#1687
Ref: https://woodruffw.github.io/zizmor/

Closes curl#15746
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.

Remove persist-credentials or change the default to false

4 participants