hydrian / TTRSS-Auth-LDAP

GitHub repository for Tiny Tiny RSS's auth_ldap plugin

https://github.com/hydrian/TTRSS-Auth-LDAP/wiki

Other

28 stars 21 forks source link

Plugin does not work anymore since tt-rss moved to PDO #26

Closed nmhuy closed 6 years ago

nmhuy commented 6 years ago

Hello,

with the plugin enabled, I get the following stack trace: PHP message: PHP Fatal error: Uncaught Error: Call to undefined method Auth_Ldap::check_password() in /tt-rss/classes/pref/prefs.php:925 Stack trace:

0 /tt-rss/include/functions.php(1091): Pref_Prefs::isdefaultpassword()

1 /tt-rss/classes/rpc.php(327): make_init_params()

2 /tt-rss/backend.php(123): RPC->sanityCheck()

3 {main}

thrown in /tt-rss/classes/pref/prefs.php on line 925

Thanks

hydrian commented 6 years ago

What is the date of your ttrss clone?

nmhuy commented 6 years ago

Around July 12th. Seems fox fixed it in ttrss.

hydrian commented 6 years ago

Looks like the project added a new required function. I'll look into it.

tsimmons commented 6 years ago

Looks like this commit fixes it: https://git.tt-rss.org/git/tt-rss/commit/d0cce0c7a431d886403632ef94add650363d4ef6

hydrian commented 6 years ago

Yup. Looks like something in the core broke the auth plugin requirements. It looks to be fixed now.

@nmhuy I would update your local ttrss core and see if you're still having a problem. If it is still happening, we'll reopen this issue.