Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Newly "created" Fixed admin account is not administrator #279

Closed
gboudreau opened this issue Oct 29, 2020 · 2 comments · Fixed by #286
Closed

Newly "created" Fixed admin account is not administrator #279

gboudreau opened this issue Oct 29, 2020 · 2 comments · Fixed by #286
Labels

Comments

@gboudreau
Copy link
Contributor

I edited sharry.conf (to enable Fixed & Internal auth, amongst other things), didn't specify a database (a new H2 db file was created), and started Sharry (using Docker).
The admin/admin account I was able to login with did not have the admin flag (I verified in the account_ table, in the H2 DB). This prevented me from being able to see the Accounts menu item, and thus create a new admin (internal) user.
I had to manually edit account_.admin to TRUE in the H2 DB.

Maybe because some incompatibility between Fixed and Internal auth..? Maybe one should not enable both together, or the admin user created in the Internal DB won't have the admin flag?

@eikek
Copy link
Owner

eikek commented Oct 29, 2020

Thank you for reporting! I look into it. Both auth mechanism should work next to each other without problems. The "fixed" one usually comes first (can be configured).

@eikek eikek added the bug label Oct 29, 2020
@bckmnn
Copy link

bckmnn commented Oct 31, 2020

I can confirm this behaviour. I also set up sharry via docker. I would not see the Invite-Generation menu when logging in as admin:admin. After connecting to the database and toggeling the flag there it worked.

eikek added a commit that referenced this issue Nov 2, 2020
The "fixed-auth" is intended to create admin accounts.

Fixes: #279
@mergify mergify bot closed this as completed in #286 Nov 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants