There are several problems that have been fixed since 7.x-2.5 release. It would be nice to get a 7.x-2.6 released. To do that we need to do two things: identify any last bugs to fix before that AND confirm that the latest 7.x-2.x-dev works well.
high-ish priority bugs/features to consider committing
#1927708: Tests that require hook invocations of CDN module's own hook implementations fail when CDN module is not enabled
#1634932: CSS aggregation cache clear broken (was: CSS aggregation with HTTPS + clear cache: CDN's CSS cache not actually cleared)
#1417616: CDN + Far Future expiration in some cases: PHP allocates all available CPU and memory
#1835186: Images linking to site are rewritten to link to the CDN (but only when using the no-www domain)
#1929918: Test coverage for image URL rewriting
#1888824: Include Drupal core's improvement to drupal_build_css_cache() to prevent unnecessary aggregates
#1740168: CDN's statistics' "touch" file links fail: "array/public://" URLs are wrong
#1859252: HTTPS check fails in environments providing HTTP_X_FORWARDED_PROTOCOL
#1926884: CDN module is not compatible with security fix in Drupal core update 7.20
In terms of giving confidence that 7.x-2.x-dev is working...there's currently only 16 users of the dev according to the usage page so it would be great if folks could comment here with feedback of whether they are using the dev and if it's working.
Comments
Comment #1
mstrelan CreditAttribution: mstrelan commentedThis is probably related to #1417616: CDN + Far Future expiration in some cases: PHP allocates all available CPU and memory, particularly comment #35.
Comment #2
gregglesLet's restate this in a positive light :)
Comment #2.0
gregglesx
Comment #3
gregglesWim, if you could review the list I created in the summary and let us know how to help you move things along that would be great! If you don't see all of these as review blockers then remove them from that list. If there are others to consider, please add them.
Comment #4
Wim LeersThanks guys :)
I'm really sorry that the CDN module has been fairly neglected (though overall it's in good shape) — this is due to my day job having been increasingly heavy in load towards Drupal feature freeze (which was one week ago now). One can only do so much… :)
I'll get this moving forward again!
Comment #5
Wim LeersThere actually already was an issue for this: #1857502: [meta] What's missing for a 2.6 release?. There, I had indicated that back in December, #1719568: CDN URLs are not properly encoded in some edge cases was the only issue we were blocked on. That is no longer the case, because that was solved (with tests, no bug) in D7, and the bug itself I cannot reproduce in D6 (others can).
#1413156: .htaccess rules for Far Future expiration: make it possible to use the Far Future feature directly in Apache, avoiding PHP is actually a new feature, so that belongs in 2.7 — created a meta issue for that as well: #1927698: [meta] 2.7 release (new features).
Updating the issue summary as I move forward.
Comment #5.0
Wim Leersx
Comment #5.1
Wim LeersRemoved #1413156: .htaccess rules for Far Future expiration: make it possible to use the Far Future feature directly in Apache, avoiding PHP, will be included in 2.7.
Comment #6
Wim LeersI just created this issue and fixed it: #1927708: Tests that require hook invocations of CDN module's own hook implementations fail when CDN module is not enabled. I want reliable test results, so I had to do that *first*.
Comment #7
Wim Leers#1634932: CSS aggregation cache clear broken (was: CSS aggregation with HTTPS + clear cache: CDN's CSS cache not actually cleared) committed.
Comment #7.0
Wim LeersAdded #1927708: Tests that require hook invocations of CDN module's own hook implementations fail when CDN module is not enabled.
Comment #8
gregglesThanks for your work, Wim. For what it's worth we've bee using the 7.x-2.x-dev from February 21 and have no problems so far.
Comment #8.0
greggles#1912438: Support SSL and Rackspace Cloud Files is not a bugfix, but a feature request.
Comment #9
Wim Leers#1929918: Test coverage for image URL rewriting created & committed. Better test coverage FTW.
Comment #10
dan.mantyla CreditAttribution: dan.mantyla commentedFYI i will be installing the dev version today, so here's another humble tester. I do have a somewhat interesting setup: using lighttpd as my server and URL rewritting, vhosts, etc. I'll let ya know if anything breaks....... (so far so good!)
Comment #11
Wim Leers#10: awesome!
Comment #11.0
Wim LeersAdding #1929918: Test coverage for image URL rewriting.
Comment #12
Wim Leers#1888824: Include Drupal core's improvement to drupal_build_css_cache() to prevent unnecessary aggregates is also important, it helps explain #1905468: Variables table is not designed to be used as a cache.
Comment #13
Wim LeersAnother minor bug bites the dust: #1740168: CDN's statistics' "touch" file links fail: "array/public://" URLs are wrong.
Comment #13.0
Wim LeersAdding #1888824: Include Drupal core's improvement to drupal_build_css_cache() to prevent unnecessary aggregates.
Comment #13.1
Wim LeersAdding #1740168: CDN's statistics' "touch" file links fail: "array/public://" URLs are wrong.
Comment #14
Wim LeersAnd another minor bug bites the dust: #1859252: HTTPS check fails in environments providing HTTP_X_FORWARDED_PROTOCOL.
Comment #14.0
Wim LeersAdding [#1859252.
Comment #15
Wim LeersThe only remaining bugs to fix are #1826816: Malformed dns-prefetch tag when using CDN's "auto-balance over multiple CDNs" feature and #1926884: CDN module is not compatible with security fix in Drupal core update 7.20.
Comment #16
mermentau CreditAttribution: mermentau commentedI have the latest 7.x-2.x-dev on 4 sites and all seems to be well. I did have to disable them after testing because of the core 7.20 issue.
Comment #16.0
mermentau CreditAttribution: mermentau commentedAdding #1826816: Malformed dns-prefetch tag when using CDN's "auto-balance over multiple CDNs" feature and #1926884: CDN module is not compatible with security fix in Drupal core update 7.20.
Comment #17
Wim LeersRemoved #1826816: Malformed dns-prefetch tag when using CDN's "auto-balance over multiple CDNs" feature, as it already was fixed. A patch for the last remaining issue — #1926884: CDN module is not compatible with security fix in Drupal core update 7.20 is available and passed local testing. I will also deploy it on http://wimleers.com to solve any remaining problems. If you can: please test it!
Comment #18
Wim Leers#1926884: CDN module is not compatible with security fix in Drupal core update 7.20 is done. That means everything is done.
2.6 releases for D6 and D7 tagged: http://drupal.org/node/1938696 & http://drupal.org/node/1938698
At last :)
Comment #19.0
(not verified) CreditAttribution: commentedRemoved #1826816: Malformed dns-prefetch tag when using CDN's "auto-balance over multiple CDNs" feature.