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

Error while creating previously deleted user #39

Open
JimboJoe opened this issue Jul 19, 2017 · 2 comments
Open

Error while creating previously deleted user #39

JimboJoe opened this issue Jul 19, 2017 · 2 comments

Comments

@JimboJoe
Copy link
Contributor

Bug originally posted on YunoHost bugtracker here.
The problem is systematically reproducible.

I tried creating a deleted user again. The account was created with error message. I can see the user in the user list and on trying to login from SSO the login is successful too.

Error:

Warning:    null, 0, null]:
Warning:   SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry 'pranav' for key 'UNIQ_8D93D64992FC23A8'
Warning: 
Warning: 
Warning: 
Warning:   [Doctrine\DBAL\Driver\PDOException]
Warning:   SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry 'pranav' for key 'UNIQ_8D93D64992FC23A8'
Warning: 
Warning: 
Warning: 
Warning:   [PDOException]
Warning:   SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry 'pranav' for key 'UNIQ_8D93D64992FC23A8'
Warning: 
Warning: 
Warning: fos:user:create [--super-admin] [--inactive] [-h|--help] [-q|--quiet] [-v|vv|vvv|--verbose] [-V|--version] [--ansi] [--no-ansi] [-n|--no-interaction] [-e|--env ENV] [--no-debug] [--] <command> <username> <email> <password>
Warning: 
Error: Script execution failed: /etc/yunohost/hooks.d/post_user_create/50-wallabag2

Analysis:
The reason is that the post user delete hook doesn't delete the user from wallabag, it only deactivates it (see here).

From what I see, there doesn't seem to be an option to delete a user from the wallabag CLI console (hence this original design choice)...

Until a solution is found, maybe we should describe this limitation in the README/documentation to advise to delete the wallabag account from wallabag (and obviously from another account!) before removing the user from YunoHost?

@JimboJoe JimboJoe added the bug label Jul 19, 2017
@lapineige
Copy link
Member

Good point, I will do it :)

(and obviously from another account!)

Why ? You can delete your own account, and I don't see what could go wrong in this way.

lapineige added a commit that referenced this issue Jul 24, 2017
@lapineige
Copy link
Member

I added the information in the README.
I keep this bug open, maybe we will find a better solution in the future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants