YunoHost / issues

General issue tracker for the YunoHost project
72 stars 8 forks source link

if an account $app exists when trying to install $app, use $app__2 instead #2393

Closed chri2 closed 2 months ago

chri2 commented 4 months ago

Describe the bug

Create an account e.g. flohmarkt and then try to install the app flohmarkt from the application menu in the webgui.

There is already a YunoHost user called flohmarkt ...

Context

Expected behavior

If there's an yunohost account $app then choose a different uid for the app like $app__2.

Logs

2024-05-16 14:46:39,108 DEBUG    moulinette.core acquire - acquiring lock...
2024-05-16 14:46:39,110 DEBUG    moulinette.core acquire - lock has been acquired
2024-05-16 14:46:39,110 DEBUG    moulinette.actionsmap process - loading python module yunohost.app took 0.000s
2024-05-16 14:46:39,111 DEBUG    moulinette.actionsmap process - processing action [3802196.49]: yunohost.app.install with args={'app': 'flohmarkt', 'label': 'undefined', 'args': 'undefined=z9flohmarkt&domain=z9.de&path=%2Fflohmarkt&init_main_permission=visitors&password_couchdb_admin=my_secret_password&mail_user=z9flohmarkt&mail_domain=link-goe.de&flohmarkt_name=my+yunohost+flohmarkt', 'no_remove_on_failure': False, 'force': False}
2024-05-16 14:46:39,164 DEBUG    yunohost.app (unknown function) - [3802196.49] Checking default branch
2024-05-16 14:46:39,702 DEBUG    yunohost.app (unknown function) - [3802196.49] Wird heruntergeladen…
2024-05-16 14:46:40,645 DEBUG    yunohost.app (unknown function) - [3802196.49] Erledigt
alexAubin commented 2 months ago

Closing because i don't understand why we should do this, having yunohost accounts named like an app is confusing as hell, yunohost accounts are supposed to correspond to human beings ...