Skip to content

ci: bound Context7 refresh transport#18

Merged
kriptoburak merged 1 commit into
mainfrom
codex/context7-transport-hardening
Jul 24, 2026
Merged

ci: bound Context7 refresh transport#18
kriptoburak merged 1 commit into
mainfrom
codex/context7-transport-hardening

Conversation

@kriptoburak

@kriptoburak kriptoburak commented Jul 24, 2026

Copy link
Copy Markdown
Member

Summary

  • restrict the Context7 refresh request to HTTPS with TLS 1.2 or newer
  • add a 10-second connection timeout and 30-second total timeout
  • preserve the existing fixed runner and 5-minute job cap

Validation

  • actionlint .github/workflows/context7-refresh.yml
  • git diff --check
  • verified the fixed runner, job timeout, and bounded curl controls
  • simplify, maintainability, and security reviews completed with no findings

OpenSSF relevance

This bounds a secret-bearing external CI request and makes failure behavior deterministic.

Note

Enforce HTTPS, TLSv1.2, and timeouts on Context7 refresh CI request

Adds --connect-timeout 10, --max-time 30, --proto '=https', and --tlsv1.2 to the curl call in context7-refresh.yml. This prevents the refresh job from hanging indefinitely and restricts the request to secure transports only.

Macroscope summarized 446e5ce.

Signed-off-by: kriptoburak <kriptoburak@users.noreply.github.com>

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your trial has ended. Reactivate Greptile to resume code reviews.

@furkanerday furkanerday left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed the complete diff, PR description, all conversation and review history, unresolved threads, repository policies, relevant contracts and tests, required checks, and commit sign-offs at 446e5ce. All required checks pass, no actionable thread remains, and I found no blocking issue.

@kriptoburak
kriptoburak merged commit 23d465b into main Jul 24, 2026
18 checks passed
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.

2 participants