Skip to content

fix(OVH): harden security - #442

Open
acolombier wants to merge 4 commits into
mixxxdj:websitefrom
acolombier:fix/ovh-security
Open

fix(OVH): harden security#442
acolombier wants to merge 4 commits into
mixxxdj:websitefrom
acolombier:fix/ovh-security

Conversation

@acolombier

@acolombier acolombier commented Aug 2, 2026

Copy link
Copy Markdown
Member
  • Harden the .htaccess config for pull request websites to protect against server-side scripts
  • Add some approval logic for low reputation accounts (first time contributors) using the preview-approved label
  • Actually allow PR to deploy they environment (currently, they deploy a copy of the main branch)

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.

Pull request overview

Hardens OVH preview deployments and adds approval gating for less-trusted contributors.

Changes:

  • Checks contributor trust and preview-approved status.
  • Deploys the pull request’s head revision.
  • Adds Apache restrictions for server-side scripts.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
.github/workflows/deploy.yml Adds preview approval and PR-ref selection logic.
.github/workflows/ovh.yaml Enforces approval and adds Apache hardening rules.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .github/workflows/deploy.yml Outdated
Comment thread .github/workflows/deploy.yml
Comment thread .github/workflows/ovh.yaml
@acolombier

Copy link
Copy Markdown
Member Author

Added a revoke logic to remove the approved deployment label upon subsequent push.

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