New version stops password resets

  • Resolved tommcgee

    (@tommcgee)


    8 years ago

    I installed the upgrade to 1.7.16 this morning, and got some reports that people who were NOT using directory authenticated accounts weren’t able to retrieve/reset their passwords.

    I tried it myself, using the account of a person who reported a problem as well as on a non-dirauth account of my own. Whether I reset it through the dashboard, through email, phpmyadmin or WP-CLI, the new passwords wouldn’t work.

    I rolled back to 1.7.15 and they work again.

Viewing 15 replies - 16 through 30 (of 30 total)
  • Plugin Author Paul Gilzow

    (@gilzow)

    Hhhmmm… not seeing any overlap in the plugins.

    If I send you two a different version, can you install it, enable WP_DEBUG and then send me the relevant lines from the log file?

    mpaskevi

    (@mpaskevi)

    Just an update, we rolled back to 1.7.15 and my external users can now login without any issue. I am sorry can’t further troubleshoot on my end at this time.

    I will be watching this issue to see if there is a resolve before I upgrade again.

    Thanks to both of you for your help!

    Thread Starter tommcgee

    (@tommcgee)

    I’d be willing to give it a try.

    Plugin Author Paul Gilzow

    (@gilzow)

    ok, copy and paste the following gist into your wpDirAuth.php file

    https://gist.github.com/gilzow/328ef3ba94f63612b208792cbc8414cb

    In your wp-config.php file, add the following lines right above where it says /* That's all, stop editing! Happy blogging. */ 

    
define('WP_DEBUG',         true);  // Turn debugging ON
define('WP_DEBUG_DISPLAY', false); // Turn forced display OFF
define('WP_DEBUG_LOG',     true);  // Turn logging to wp-content/debug.log ON

    Attempt to login using the local account that is failing when it shouldn’t. In your debug.log file (it’s location is in \wp-content\ by default) send me the lines that include WPDIRAUTH. The lines should look similar to

    [28-Nov-2016 16:38:52 UTC] WPDIRAUTH: At line number 1183

    Or if it’s easier, you can send me the whole log file.

    Plugin Author Paul Gilzow

    (@gilzow)

    @tommcgee, were you ever able to install the altered version of wpDirAuth? If so, can you send me the output from the log files so I can try and troubleshoot the issue?

    Thread Starter tommcgee

    (@tommcgee)

    No, not yet. Our sites are for a university, and I don’t want to risk anything going down during finals. But it’s definitely on my schedule for when the students clear out.

    Plugin Author Paul Gilzow

    (@gilzow)

    Totally understand. Our students finished last week, so this week should be more relaxed.

    Thread Starter tommcgee

    (@tommcgee)

    Hi gilzow, thanks for holding. I just went through the steps you outlined, and my error log doesn’t show anything directly relevant. This is what’s there: 

    [04-Jan-2017 19:02:37] PHP Warning:  include(/var/www/html/wp-content/advanced-cache.php): failed to open stream: No such file or directory in /var/www/html/wp-settings.php on line 84
[04-Jan-2017 19:02:37] PHP Warning:  include(/var/www/html/wp-content/advanced-cache.php): failed to open stream: No such file or directory in /var/www/html/wp-settings.php on line 84
[04-Jan-2017 19:02:37] PHP Warning:  include(): Failed opening '/var/www/html/wp-content/advanced-cache.php' for inclusion (include_path='.:/usr/share/pear:/usr/share/php') in /var/www/html/wp-settings.php on line 84
[04-Jan-2017 19:02:37] PHP Warning:  include(): Failed opening '/var/www/html/wp-content/advanced-cache.php' for inclusion (include_path='.:/usr/share/pear:/usr/share/php') in /var/www/html/wp-settings.php on line 84
[04-Jan-2017 19:02:37] PHP Warning:  include(/var/www/html/wp-content/advanced-cache.php): failed to open stream: No such file or directory in /var/www/html/wp-settings.php on line 84
[04-Jan-2017 19:02:37] PHP Warning:  include(): Failed opening '/var/www/html/wp-content/advanced-cache.php' for inclusion (include_path='.:/usr/share/pear:/usr/share/php') in /var/www/html/wp-settings.php on line 84
[04-Jan-2017 19:02:37] PHP Warning:  include(/var/www/html/wp-content/advanced-cache.php): failed to open stream: No such file or directory in /var/www/html/wp-settings.php on line 84
[04-Jan-2017 19:02:37] PHP Warning:  include(): Failed opening '/var/www/html/wp-content/advanced-cache.php' for inclusion (include_path='.:/usr/share/pear:/usr/share/php') in /var/www/html/wp-settings.php on line 84
[04-Jan-2017 19:02:37] PHP Notice:  load_plugin_textdomain was called with an argument that is <strong>deprecated</strong> since version 2.7.0 with no alternative available. in /var/www/html/wp-includes/functions.php on line 3978
[04-Jan-2017 19:02:37] PHP Notice:  load_plugin_textdomain was called with an argument that is <strong>deprecated</strong> since version 2.7.0 with no alternative available. in /var/www/html/wp-includes/functions.php on line 3978
    Plugin Author Paul Gilzow

    (@gilzow)

    ok, that’s doubly odd as none of those entries were generated by wpDirAuth. I would suggest cleaning up the issues with advanced-cache.php, though I don’t think it would cause the issue you are seeing with wpDirAuth.

    Something is definitely not right since, at a minimum, it should have logged the user account that was going to be authenticated (from line 1183) and whether the user is an AD-authed account or not (line 1185). Those lines should have been processed unless the username or password fields were blank (returns at 1160 and 1166).

    Thread Starter tommcgee

    (@tommcgee)

    Sorry, I’m a little fuzzy around the edges from two weeks away from the office. I was working on the wrong machine. Here’s the log entries you’re looking for: 

    [05-Jan-2017 13:47:57] WPDIRAUTH: At line number 1183 
account that needs to be authed 
'(redacted)'
[05-Jan-2017 13:47:57] WPDIRAUTH: At line number 1185 
is (redacted) AD authed? 
''
[05-Jan-2017 13:47:57] WPDIRAUTH: At line number 1273 
autoregister enabled? 
false
[05-Jan-2017 13:47:57] WPDIRAUTH: At line number 1274 
what is $login? 
WP_User::__set_state(array(
   'data' => 
  stdClass::__set_state(array(
     'ID' => '1',
     'user_login' => '(redacted)',
     'user_pass' => '$P$BQ00ySkNnXv7OI0CkN0lfQqD0dmfJX.',
     'user_nicename' => '(redacted)',
     'user_email' => '(redacted)',
     'user_url' => '',
     'user_registered' => '2012-05-09 16:31:48',
     'user_activation_key' => '',
     'user_status' => '0',
     'display_name' => 'Tom McGee',
     'spam' => '0',
     'deleted' => '0',
  )),
   'ID' => 1,
   'caps' => 
  array (
    'subscriber' => true,
  ),
   'cap_key' => 'wp_ep1_capabilities',
   'roles' => 
  array (
    0 => 'subscriber',
  ),
   'allcaps' => 
  array (
    'level_0' => true,
    'read' => true,
    'read_ai1ec_event' => true,
    'subscriber' => true,
  ),
   'filter' => NULL,
))
[05-Jan-2017 13:47:57] WPDIRAUTH: At line number 1279 
dealing with a local account 
NULL
[05-Jan-2017 13:47:57] PHP Warning:  is_a() expects exactly 2 parameters, 3 given in /var/www/html/wp-content/plugins/wpdirauth/wpDirAuth.php on line 1283
[05-Jan-2017 13:47:57] WPDIRAUTH: At line number 1290 
results from wp_check_password after failing line 1283 
true
    Plugin Author Paul Gilzow

    (@gilzow)

    that looks better. What version of PHP are you on? I’m going to bet something less than 5.3.9?

    Thread Starter tommcgee

    (@tommcgee)

    And you would have won the bet! If you want to work through specifics drop me a line at thomas -dot- mcgee -at- shu.edu.

    Plugin Author Paul Gilzow

    (@gilzow)

    email on its way!

    Plugin Author Paul Gilzow

    (@gilzow)

    version 1.7.17 has been released which corrects this issue. Many thanks to @tommcgee for working with me to troubleshoot this problem!

    @mpaskevi, can you update and see if things are now working for you as they should?

    Thread Starter tommcgee

    (@tommcgee)

    Works for me! Thanks again.

Viewing 15 replies - 16 through 30 (of 30 total)
  • The topic ‘New version stops password resets’ is closed to new replies.