CSS aggregation breaks urls with domains starting with base-domain.
I.e. an aggregated css from "http://mydomain/" which has resources/urls in it pointing to "http://mydomain:81/" are broken and converted to ":81/" - which results in broken aggregated and rendered css. This only occurs in some edge cases (testing CDN/different domains for serving css mapped locally with nginx running on a different port number).

CommentFileSizeAuthor
#1 1089558_CORE_common_inc__CSS_aggregation_port_number_fix.patch725 bytesdanielnolde

Comments

danielnolde’s picture

danielnolde commented
StatusFileSize
new 1089558_CORE_common_inc__CSS_aggregation_port_number_fix.patch725 bytes

This bug is easily fixed with the attached patch. Please review and ... commit ;)

steven jones’s picture

steven jones commented

Subscribe.

I'm wondering if JS aggregation is affected too?

steven jones’s picture

steven jones commented

It doesn't apply to JS aggregation.

john_b’s picture

john_b commented

Support please.

I applied patch to 7.7. redirecting from mydomain.com to mydomain.co.uk for css files. chowned directory structure of mydomain.co.uk to same user as main domain, and disabled cPanel's open_basedir for relevant account.

Still I get 'The specified file temporary://file4RHVEq could not be copied, because the destination directory is not properly configured. This may be caused by a problem with file or directory permissions. More information is available in the system log.'
Maybe the answer should be obvious?

Status: Needs review » Closed (outdated)

Automatically closed because Drupal 7 security and bugfix support has ended as of 5 January 2025. If the issue verifiably applies to later versions, please reopen with details and update the version.