git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH v2] ci: pin actions in the main workflow to full commit IDs

From
Junio C Hamano <gitster@pobox.com>
Date
Oct 26, 2026, 18:20 UTC
Message-ID
<xmqq7bj4qq6i.fsf@gitster.g>
In-Reply-To
<67057c2277e38a675ffbe0206bf249c24efdd6b6.1793038091.git.ttaylorr@openai.com>
Taylor Blau <ttaylorr@openai.com> writes:
Show 18 quoted lines
> Our main CI workflow refers to actions by version tags. These tags can
> move, changing the action code we run without any change to our
> repository. If an attacker gains access to an action's repository,
> they can move an existing tag to a malicious revision and have our
> workflow pick it up automatically.
>
> Pin these actions to full commit IDs so that moving a tag cannot
> substitute a different action revision. Updating the actions will
> require an explicit change to our workflow, including when picking up
> bug fixes and security fixes within a major version.
>
> Retain the version names in comments to keep the references readable
> and make future updates easier to review.
>
> Signed-off-by: Taylor Blau <ttaylorr@openai.com>
> ---
>  .github/workflows/main.yml | 64 +++++++++++++++++++-------------------
>  1 file changed, 32 insertions(+), 32 deletions(-)

With these changes, the proposal no longer appears to bend the public version to suit a single employee's environment. We might even want to strengthen the reasoning by adding "becoming the de facto security standard", borrowing from Ben Knoble's comment in the previous round.

Thanks.  Queued.
Previous: Taylor Blau
Message 7 of 7 in “ci: pin actions in the main workflow to full commit IDs”
  1. ci: pin actions in the main workflow to full commit IDsTaylor Blau, Oct 23, 2026
  2. Ben KnobleOct 23, 2026
  3. Junio C HamanoOct 23, 2026
  4. Taylor BlauOct 24, 2026
  5. Junio C HamanoOct 24, 2026
  6. ci: pin actions in the main workflow to full commit IDsTaylor Blau, Oct 26, 2026
  7. Junio C HamanoOct 26, 2026

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.