Add ability to disable unbounce plugin's debug logging? You're flooding the log!

  • Resolved lkraav

    (@lkraav)


    9 years ago

    Good day. I’d like to request a reconsideration of your debug logging mechanism. It’s common practice to have WP_DEBUG and WP_DEBUG_LOG permanently ON both in development and production, because it’s the only way to catch issues. During the last decade of working with WP, I’ve yet to see any other plugin completely flood debug.log like the Unbounce plugin does, which leads me to believe the developers here have misunderstood a few aspects about WordPress debug logging.

    This is way too harsh:

    UBConfig.php:

    public static function debug_loggging_enabled() {
    return WP_DEBUG || WP_DEBUG_LOG || UBConfig::remote_debug_logging_enabled();
  }

    Could this be altered with an apply_filters() call for the result, so I could filter it always OFF with an mu-plugin but still keep the rest of my debug.log?

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

Viewing 5 replies - 1 through 5 (of 5 total)
  • Plugin Author Unbounce

    (@unbouncewordpress)

    Thank you for reaching out. We appreciate your insight into the WordPress debug logging. We have added an issue to our backlog so we can look at improving this in the future.

    Thread Starter lkraav

    (@lkraav)

    Any time soon? Every minor version update causes trouble.

    Thread Starter lkraav

    (@lkraav)

    Easiest and low impact way to do this is to introduce a constant like UB_DEBUG_LOG.

    First, set it yourself somewhere but only if it’s not already defined:

    if ( ! defined( "UB_DEBUG_LOG" ) ) {
  define( "UB_DEBUG_LOG", true );
}

    Then modify debug_logging_enabled():

    return UB_DEBUG_LOG && ( WP_DEBUG || WP_DEBUG_LOG || UBConfig::remote_debug_logging_enabled(); )

    This allows me to stick define( “UB_DEBUG_LOG”, false ) in my wp-config.php.

    Any objections?

    Plugin Author Unbounce

    (@unbouncewordpress)

    We have prioritized fixing this issue and we will update this thread as soon as it’s complete.

    Thread Starter lkraav

    (@lkraav)

    I see it was released in a minor release a short while ago. Thanks for taking care here!

Viewing 5 replies - 1 through 5 (of 5 total)
  • The topic ‘Add ability to disable unbounce plugin's debug logging? You're flooding the log!’ is closed to new replies.