Skip to content

Harden site build and search boundaries - #105

Merged
VatsalSy merged 3 commits into
mainfrom
local-branch/security-findings-backlog
Jul 29, 2026
Merged

Harden site build and search boundaries#105
VatsalSy merged 3 commits into
mainfrom
local-branch/security-findings-backlog

Conversation

@VatsalSy

Copy link
Copy Markdown
Member

Summary

  • harden the Jekyll and npm build paths identified during the 44-finding Codex Security triage
  • constrain search indexing and client-side rendering boundaries, including safer URL and content handling
  • tighten workflow, lint, dependency, CSS, and legacy-page behaviour
  • pin Ruby 3.4.10 and declare the base64 and csv gems required by Jekyll on Ruby 3.4

Why

The security scan surfaced a mix of actionable hardening gaps, stale assumptions, and false positives. The actionable cases were fixed; the remaining findings were reviewed and closed with a decision. Running the repository under its pinned Ruby version also exposed that Jekyll relied on standard-library gems that Ruby 3.4 no longer installs by default.

Impact

The public site keeps its existing behaviour while reducing unsafe search/rendering paths and making local and CI builds reproducible with the pinned toolchain.

Validation

  • clean full Jekyll build with Ruby 3.4.10 and Bundler 2.5.23
  • 13 Jest suites / 85 tests passed
  • JavaScript, CSS, and Markdown lint passed
  • content rules and security trigger parity passed
  • Ruby syntax and GitHub Actions YAML validation passed
  • npm audit --omit=dev: 0 vulnerabilities
  • all 44 Codex Security findings triaged and marked done

VatsalSy added 3 commits July 28, 2026 22:10
Pin browser and build dependencies, constrain untrusted search URLs, and fail closed on local tooling errors. Restrict Actions tokens, move Pages credentials to deploy jobs, upgrade Ruby, and fix the affected UI and packaging regressions.
Add base64 and csv as explicit dependencies now that Ruby 3.4 no longer ships them as default gems. This keeps the pinned Jekyll build runnable locally and in CI.
Write the Pages archive to the explicit GitHub workspace path so tar and gzip agree across runners. Drop the unused GNU-only hard-link flag to keep local reproduction portable.
@VatsalSy
VatsalSy marked this pull request as ready for review July 29, 2026 07:56
Copilot AI review requested due to automatic review settings July 29, 2026 07:56

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@VatsalSy
VatsalSy merged commit 9ec9aa4 into main Jul 29, 2026
5 checks passed
@VatsalSy
VatsalSy deleted the local-branch/security-findings-backlog branch July 29, 2026 07:56
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