Forum Replies Created

Viewing 6 replies - 1 through 6 (of 6 total)
  • Thread Starter gohanman

    (@gohanman)

    FWIW, I did look at the documentation before posting. What’s distinctly not helpful about the documentation is that it also does not say which version of OpenSSL is supported. Saying “more than 5 years old” is way more ambiguous than listing a minimum version number. Also, OpenSSL 1.0.1 was released a bit over 6 years ago.

    Thread Starter gohanman

    (@gohanman)

    The suggestion has less to do with honesty and more to do with failing in a logged manner.

    If the environment is configured with display_errors off using trigger_error would provide even less information with a blank white screen instead of “Go away script kiddies”. But even if display_errors is on the point of the suggestion is to put it at the top of the script to prevent access. There’s nothing on the call stack to leak into display.

    Thread Starter gohanman

    (@gohanman)

    Yeah. The problem actually stemmed from the security advice here: https://codex.ww.wp.xz.cn/Writing_a_Plugin#Plugin_Files

    I wrote

    
    defined( 'ABS_PATH' ) or die( 'No script kiddies please!' );
    

    instead of

    
    defined( 'ABSPATH' ) or die( 'No script kiddies please!' );
    

    but die() isn’t really a fatal error. It just prints to STDOUT (which presumably gets eaten by output buffering somewhere) and exits.

    Something like this would probably be a better solution.

    
    if (!defined('ABSPATH')) {
        trigger_error('Invalid call state', E_USER_ERROR);
    }
    

    I’m also experiencing this in the latest version / 8.1.3.

    Thread Starter gohanman

    (@gohanman)

    Version control! I never had WordPress itself under VC, but there was an old .svn folder farther up the directory tree (an artifact of hosting multiple domains under the same root directory). It took a bit of digging through the code to figure out what paths it was checking for version control.

    From what I saw tracing through this, it looks to me like automatic background updates won’t run under any of the following conditions:
    a) both cURL and streams are unavailable
    b) cURL and/or streams do not have SSL support
    c) WordPress appears to be under version control

    IMO it would be helpful to note somewhere which of these checks is failing. I realize this isn’t useful information to all users and showing it prominently would confuse more than it helps, but having the details available somewhere would have been quicker than familiarizing myself with WP’s HTTP abstractions to figure out what was happening.

    Thread Starter gohanman

    (@gohanman)

    It’s entirely possible I’m in the wrong spot. I thought the plugin was responsible for adding a WYSIWYG editor. I may have mixed it up since both are labeled “TinyMCE”. Sorry about the mix up.

    The TM Lunch Menu plugin did have WYSWYG fields prior to WP 3.9. But if your plugin has nothing to do with general WP’s general WYSWYG editor, this really isn’t the right place for debugging.

Viewing 6 replies - 1 through 6 (of 6 total)