Skip to content
This repository was archived by the owner on Oct 29, 2021. It is now read-only.

User recreated every time (and lose assigned groups) with new version Hrun #22

Open
daaain opened this issue Sep 30, 2014 · 5 comments
Open
Labels

Comments

@daaain
Copy link

daaain commented Sep 30, 2014

Just tried the new Dokuwiki version (2014-09-29 "Hrun") on my local machine and (possibly due to the changes in the auth methods) now if I log out and log back in, each time there's a new entry generated in users.auth.php. This means that users lose any usergroup (like admin) other than the one assigned by default.

@kettari
Copy link
Owner

kettari commented Sep 30, 2014

This issue only valid for the "Hrun" version? For previous versions it's OK?

@InfernoZeus
Copy link

I've noticed similar behaviour with previous versions (although I don't use
user groups do can't confirm that), but I do see already existing users who
log in again moved to the bottom of the use list.
On 30 Sep 2014 18:32, "Andy" notifications@github.com wrote:

This issue only valid for the "Hrun" version? For previous versios it's OK?


Reply to this email directly or view it on GitHub
#22 (comment).

@daaain
Copy link
Author

daaain commented Sep 30, 2014

Yes, when I revert back to the current, non-hotfixed version (2014-05-05a "Ponder Stibbons") it does work again. I haven't tried the hotfixed one (2014-05-05b "Ponder Stibbons") yet, do you want me to give it a go?

@InfernoZeus I've also noticed that the lines jump around in the file, but the user group settings are retained. You could argue that's a feature not a bug as this way you can see who logged in last :)

But in the case of the latest version a new line gets added for each login, so effectively it seems that the existing user ID is somehow not picked up and a new one is created every time.

@kettari
Copy link
Owner

kettari commented Sep 30, 2014

Could you please propose a patch?

@daaain
Copy link
Author

daaain commented Sep 30, 2014

I'm not confident enough with Dokuwiki internals to be able to solve it, but more importantly I won't have time to really dive into this until this Friday the earliest – at which point I might be able to at least try to help to narrow the problem down.

What I did notice when looking at git diff after the upgrade is that it seems $requireGroups parameter seems to have been added to a lot of auth related method signatures.

@kettari kettari added bug and removed bug labels Sep 30, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants