Skip to content

[fix]: update Content Security Policy in .htaccess to include new domains#3979

Merged
Duansg merged 1 commit into
apache:masterfrom
zqr10159:fix-htaccess
Jan 16, 2026
Merged

[fix]: update Content Security Policy in .htaccess to include new domains#3979
Duansg merged 1 commit into
apache:masterfrom
zqr10159:fix-htaccess

Conversation

@zqr10159

Copy link
Copy Markdown
Member

What's changed?

Checklist

  • I have read the Contributing Guide
  • I have written the necessary doc or comment.
  • I have added the necessary unit tests and all cases have passed.

Add or update API

  • I have added the necessary e2e tests and all cases have passed.

Copilot AI review requested due to automatic review settings January 16, 2026 12:45
@github-actions github-actions Bot added the doc Improvements or additions to documentation label Jan 16, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Content Security Policy (CSP) configuration in the .htaccess file to properly define security policies for third-party services used by the HertzBeat documentation site.

Changes:

  • Replaces the environment variable approach with direct CSP header configuration using Apache's mod_headers module
  • Adds comprehensive CSP directives covering multiple security aspects including default sources, frames, workers, images, fonts, and objects
  • Includes necessary domains for Algolia search, Kapa.ai widget, Apache infrastructure, and other third-party services

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread home/static/.htaccess
Comment thread home/static/.htaccess
Comment thread home/static/.htaccess
Comment thread home/static/.htaccess
Comment thread home/static/.htaccess
@Duansg Duansg merged commit 43332f4 into apache:master Jan 16, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants