Skip to content

fix: Restrict sdk-py integration secrets to trusted refs - #8310

Open
langsmith-fleet[bot] wants to merge 1 commit into
mainfrom
fix/sdk-integration-ci-secrets
Open

fix: Restrict sdk-py integration secrets to trusted refs#8310
langsmith-fleet[bot] wants to merge 1 commit into
mainfrom
fix/sdk-integration-ci-secrets

Conversation

@langsmith-fleet

Copy link
Copy Markdown
Contributor

Summary

  • Disable the sdk-py-integration-test reusable workflow on pull_request events so PR-controlled integration files cannot run with LANGSMITH_API_KEY in the container environment.
  • Replace secrets: inherit for this job with explicit named secrets required by _sdk_integration_test.yml.
  • Keep the suite available for trusted refs such as pushes to main and manual workflow dispatches.

Security

Addresses Corridor category 1 findings:

Testing

  • Parsed .github/workflows/ci.yml and .github/workflows/_sdk_integration_test.yml with Ruby YAML.
  • Ran git diff --cached --check before commit.

@jkennedyvz John Kennedy (jkennedyvz) changed the title Restrict sdk-py integration secrets to trusted refs fix: Restrict sdk-py integration secrets to trusted refs Jul 9, 2026
@jkennedyvz
John Kennedy (jkennedyvz) enabled auto-merge (squash) July 9, 2026 03:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant