Skip to content

fix(ci): pin Kong/kong-license to a commit SHA instead of @master - #2178

Merged
Prashansa-K merged 2 commits into
mainfrom
fix/KAG-9553-pin-kong-license-sha
Aug 5, 2026
Merged

fix(ci): pin Kong/kong-license to a commit SHA instead of @master#2178
Prashansa-K merged 2 commits into
mainfrom
fix/KAG-9553-pin-kong-license-sha

Conversation

@cv1002

@cv1002 cv1002 commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Kong/kong-license's master branch has no branch protection, so the unpinned Kong/kong-license@master reference in integration-ai-gateway.yaml lets anyone with push access to that repo run code in this repo's CI, which has access to secrets.
  • Pins the action to the current master tip (ddccb487a0b319cef85f348b16a1ad2b6d9c1df7), the same SHA already used by other Kong repos (e.g. kong-ee) and by this repo's own other workflows.

Kong/kong-license's master branch has no branch protection, so an
unpinned @master reference lets anyone with push access to that repo
run code in this repo's CI, which has access to secrets.

Pin to the current master tip (ddccb487a0b319cef85f348b16a1ad2b6d9c1df7),
the same SHA already used by other Kong repos such as kong-ee.

Ref: KAG-9553
@CLAassistant

CLAassistant commented Aug 3, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@codecov-commenter

codecov-commenter commented Aug 3, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 35.92%. Comparing base (39f1ee2) to head (83d0d26).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2178   +/-   ##
=======================================
  Coverage   35.92%   35.92%           
=======================================
  Files          82       82           
  Lines        7832     7832           
=======================================
  Hits         2814     2814           
  Misses       4753     4753           
  Partials      265      265           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Prashansa-K
Prashansa-K merged commit 4657ee3 into main Aug 5, 2026
51 checks passed
@Prashansa-K
Prashansa-K deleted the fix/KAG-9553-pin-kong-license-sha branch August 5, 2026 07:19
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.

7 participants