• Resolved onetrev

    (@onetrev)


    Just thought I should mention this while you are in the process of updating the plugin to a new version. Maybe the new CSS minifier in there will resolve this as well.

    As soon as I turn on “Optimize CSS Code” I don’t get the white screen people speak of, but instead the page load fails and returns “The connection was reset” in all browsers tested.

    This seems to only happen on my local machine running a fairly recent version of WAMP. With legacy CSS minification implemented in htaccess all is fine, so it’s no big deal, just wanted to put this info out there.

    Thanks again for the plugin it’s awesome!!

    https://www.ads-software.com/plugins/autoptimize/

Viewing 9 replies - 16 through 24 (of 24 total)
  • Plugin Author Frank Goossens

    (@futtta)

    very interesting indeed Franck160!

    Could you guys try the apache config change as described here instead of your “quick (and dirty) patch”?

    in httpd.conf:

    <IfModule mpm_winnt_module>
       ThreadStackSize 8*1024*1024
    </IfModule>

    To Futtta and Franck160,

    ThreadStackSize is definitely solving the problem!

    It’s wow again ??

    Now, default theme (twentyfourteen) is now working perfectly on my localhost with everything minified (including the css file) !

    As Include conf/extra/httpd-mpm.conf was already uncommented in the httpd.conf file, the change as been inserted as follows in httpd-mpm.conf instead:

    # WinNT MPM
    # ThreadsPerChild: constant number of worker threads in the server process
    # MaxConnectionsPerChild: maximum number of connections a server process serves
    # Adding ThreadStackSize on 2014-01-04
    <IfModule mpm_winnt_module>
        ThreadStackSize    8388608
        ThreadsPerChild        150
        MaxConnectionsPerChild   0
    </IfModule>

    8388608 being equivalent to 8*1024*1024… (see How do I increase the stack size for Apache running under Windows 7?).

    Admittedly, the contribution of Franck160 was a nice (and a dirty) one, but changing the ThreadStackSize is more elegant.

    Now going back to my ongoing project with a smile…

    Thanks to both of you and have a nice day!

    ?? Yes! you found the correct workout ??

    Note that in my case,
    WP – 3.8
    Autoptimize 1.8
    W7
    Apache 2.2.21
    PHP 5.3.13
    MySQL 5.5.16

    ThreadStackSize 8*1024*1024 didn’t work… but ThreadStackSize 8388608 did ??

    So complete solution is:
    httpd.conf:

    # Server-pool management (MPM specific)
    Include conf/extra/httpd-mpm.conf

    and into conf/extra/httpd-mpm.conf

    <IfModule mpm_winnt_module>
        ThreadStackSize 8388608
    </IfModule>

    @zakoops houps sorry, I didn’t saw your answer;).

    @futtta thanks for your interrest in our answer & support! ??

    Best regards – and happy new year ??

    Plugin Author Frank Goossens

    (@futtta)

    ThreadStackSize 8*1024*1024 didn’t work… but ThreadStackSize 8388608 did ??

    confirmed, same here. this is pretty low-level sssht. I’ll add an entry in the FAQ explaining how Apache on Windows might mess up!

    Thanks for all the help guys, but I hope I never have to dive into such mess again really ??

    Plugin Author Frank Goossens

    (@futtta)

    I’ve been looking into this even more after experiencing a similar problem on Linux (which has a default stacksize of 8MB) when trying to optimize All-in-one-buttons’ CSS.

    I’ve created a pretty detailed bug-report over at YUI-CSS-compressor-PHP-port’s Github, let’s hope the developer is smarter then I am ??

    Plugin Author Frank Goossens

    (@futtta)

    did some trial and (a lot of) error and came up with;

    $css = preg_replace_callback('/(^[^\{]*)(\s(:.*\{))/', array($this, 'replace_colon'), $css);

    as an alternative, much simpler regex on line 294. could you guys try this one out?

    Thread Starter onetrev

    (@onetrev)

    Hmm, did you mean line 291? If so, the new regex DID solve this for me!!

    And while I’m here thanks to all for working on this. I started a beast apparently, but I’m glad it wasn’t just something weird on my machine.

    Plugin Author Frank Goossens

    (@futtta)

    OK, applied this fix on trunk (the upcoming 1.8), which can be downloaded here.

Viewing 9 replies - 16 through 24 (of 24 total)
  • The topic ‘Connection reset – legacy filter required’ is closed to new replies.