13
39
Fork
You've already forked forte
10

cdnhost #43

Merged
fortified merged 5 commits from wago/forte:cdnhost into dev 2026年05月12日 02:54:15 +02:00
Contributor
Copy link

adds CDN host option with corresponding csp headers
if App::$config['system']['script_path_use_cdn'] is present in .htconfig.php it returns that string in function script_path() (src/Lib/Head.php)
also adds cdn host to $cspsettings array under "security headers" in boot.php

adds CDN host option with corresponding csp headers if App::$config['system']['script_path_use_cdn'] is present in .htconfig.php it returns that string in function script_path() (src/Lib/Head.php) also adds cdn host to $cspsettings array under "security headers" in boot.php
Collaborator
Copy link

@wago
I would like to see the setting with the comments in install/htconfig.sample.php, with the setting commented out by default. The code itself should be good without comments then.

Can't comment on the logic. I guess if it works for you, it should be fine.

@wago I would like to see the setting with the comments in install/htconfig.sample.php, with the setting commented out by default. The code itself should be good without comments then. Can't comment on the logic. I guess if it works for you, it should be fine.
Author
Contributor
Copy link

ok - updated the sample config. it's working with CDN on my Forte instance.

ok - updated the sample config. it's working with CDN on my Forte instance.
Author
Contributor
Copy link

please wait a bit to merge, i want to verify there is no issue with cookies

please wait a bit to merge, i want to verify there is no issue with cookies

Bump. If this is working ok for you, will merge. If not, let's close it.

Bump. If this is working ok for you, will merge. If not, let's close it.

Oh never mind. As long as it's optional, I don't see any harm in putting it in a config. Will merge.

Oh never mind. As long as it's optional, I don't see any harm in putting it in a config. Will merge.
Sign in to join this conversation.
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
fortified/forte!43
Reference in a new issue
fortified/forte
No description provided.
Delete branch "wago/forte:cdnhost"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?