-
Notifications
You must be signed in to change notification settings - Fork 665
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Subresource Integrity failures when using ASSET_URL #1426
Labels
Comments
Something else I've noticed is vite is configured to not have hashes in filenames,presumably to keep git cleaner but assets will be non-cache-busting. Not sure if there is a way to get vite to handle it, or createAssetPathsUsing appending Vite::manifestHash() to the query string? |
mmachatschek
added a commit
to mmachatschek/horizon
that referenced
this issue
Apr 24, 2024
…rity check Fixes laravel#1425 Fixes laravel#1426
@mnightingale PR#1427 should solve your issue |
I have the same issue, thank you |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Horizon Version
5.24.3
Laravel Version
10.48.9
PHP Version
8.3.6
Redis Driver
PhpRedis
Redis Version
6.0.2
Database Driver & Version
No response
Description
Related #1421
I have a deployment using a CDN via
ASSET_URL=https:://cdn.domain.com
but I'm running into CORs issues.Chrome reports:
CORs is confusing but I resolve for the main site using AppServiceProvider with the following, but all my customisations via the Vite facade appear to be ignored within horizon including createAssetPathsUsing which I hoped could be an alternative to exclude horizon paths from using the CDN altogether.
I'm not sure how to fix it but it looks like
horizon/resources/views/layout.blade.php
Lines 1 to 21 in bf3c4a8
For now I've reverted to 5.24.2 which doesn't have integrity attributes so loads without issue.
Steps To Reproduce
Access assets via a different domain, i.e ASSET_URL=https://cdn.domain.com and try to access the horizon dashboard.
The text was updated successfully, but these errors were encountered: