jasperweyne / PiwigoOpenIdConnect

A plugin for Piwigo to enable OpenID Connect based login
Apache License 2.0
10 stars 8 forks source link

fix: pre plugin installation users not being connected #12

Open ashemsay opened 2 years ago

ashemsay commented 2 years ago

I installed and configured the plugin but when I tried to log back in with my account it did not work. It turns out oidc users can't be logged in if they where already registered before the installation of the plugin. That is a big issue for me because:

I added a test to verify if a user with the same username already exists before registering, if so don't try to create a new user but only handle the oidc table part.

netchild commented 1 year ago

I stumbled upon the same problem with piwigo 13. I tried your patch, but it needs a bit of work. The users table name is not fixed, it can be prefixed. When I hardcoded my table name, it sort of worked, but overwrote the username with an UID (I have "username" in the "preferred username" part of the plugin config). I fixed that manually in the DB and the SSO seems to work.