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

Linter rebased #27

Merged
merged 57 commits into from
Jan 18, 2022
Merged

Linter rebased #27

merged 57 commits into from
Jan 18, 2022

Conversation

Gredin67
Copy link
Collaborator

Linter rebased on master to prepare for #24

@Gredin67
Copy link
Collaborator Author

!testme

@yunohost-bot
Copy link
Contributor

😜
Test Badge

@MayeulC
Copy link
Collaborator

MayeulC commented Oct 28, 2021

 > Reinstalling after removal.

Running: yunohost app install --force /app_folder -a "# mautrix_whatsapp manifest parameters&port=8449&synapsenumber=1&botname=signalbot&encryption=0&botadmin=package_checker&botusers=@johndoe:synapsedomain.tld&domain=sub.domain.tld&path=&admin=&is_public=&"
1410 INFO Installing mautrix_signal...
2711 INFO [....................] > Validating installation parameters...
2850 INFO [....................] > Finding an available port...
3284 INFO [+...................] > Storing installation settings...
5379 INFO [#++++++++++++++.....] > Installing dependencies...
53976 WARNING head: cannot open '/etc/ssl/certs/java/cacerts' for reading: No such file or directory
61113 INFO '/etc/apt/trusted.gpg.d/mautrix_signal.asc' wasn't deleted because it doesn't exist.
65600 INFO [###############.....] > Creating a PostgreSQL database...
66599 WARNING The user mautrix_signal exists, please provide his password
68131 ERROR Unable to install mautrix_signal: An error occurred inside the app installation script
68136 INFO The operation 'Install the 'mautrix_signal' app' could not be completed. Please share the full log of this operation using the command 'yunohost log share 20211028-032859-app_install-mautrix_signal' to get help
68172 WARNING Here's an extract of the logs before the crash. It might help debugging the error:
68172 INFO DEBUG - + db_name=mautrix_signal
68172 INFO DEBUG - + db_user=mautrix_signal
68173 INFO DEBUG - + ynh_app_setting_set --app=mautrix_signal --key=db_name --value=mautrix_signal
68173 INFO DEBUG - + [[ db_name =~ (unprotected|protected|skipped)_ ]]
68173 INFO DEBUG - + ynh_app_setting set mautrix_signal db_name mautrix_signal
68173 INFO DEBUG - + ynh_psql_test_if_first_run
68174 INFO DEBUG - + dpkg --list
68174 INFO DEBUG - + grep -q 'ii  postgresql-11'
68174 INFO DEBUG - + '[' -e /etc/postgresql/11 ']'
68174 INFO DEBUG - + systemctl is-active postgresql@11-main -q
68174 INFO DEBUG - + systemctl is-enabled postgresql -q
68175 INFO DEBUG - + '[' '!' -f /etc/yunohost/psql ']'
68175 INFO DEBUG - + ynh_psql_setup_db --db_user=mautrix_signal --db_name=mautrix_signal
68175 INFO DEBUG - + db_pwd=
68175 INFO DEBUG - + ynh_psql_user_exists --user=mautrix_signal
68175 INFO DEBUG - + grep --quiet mautrix_signal
68176 INFO DEBUG - ++ cat /etc/yunohost/psql
68176 INFO DEBUG - + sudo --login --user=postgres PGUSER=postgres PGPASSWORD=********** psql -tAc 'SELECT rolname FROM pg_roles WHERE rolname='\''mautrix_signal'\'';'
68176 INFO DEBUG - + return 0
68176 INFO DEBUG - + '[' -z ']'
68176 INFO DEBUG - + ynh_die '--message=The user mautrix_signal exists, please provide his password'

Either we should better clean up during removal, or we should better handle that case during install. I'm leaning towards the former.

@Gredin67
Copy link
Collaborator Author

!testme

@yunohost-bot
Copy link
Contributor

🎠
Test Badge

@Gredin67 Gredin67 mentioned this pull request Jan 17, 2022
5 tasks
@Gredin67 Gredin67 merged commit 36a2c4a into testing Jan 18, 2022
Gredin67 added a commit that referenced this pull request Jan 25, 2022
* Upgrade to v0.2.2

* do not specify signald version

* Add support for end-to-bridge encryption (#37) (deactivated by default)

* Linter and sync whatsapp_ynh (#27)

* prepare for add to app list

* restore signald user data folder

Co-authored-by: ericgaspar <[email protected]>

* python3-dev

* no python-olm

* check upgrade from 0.2.0

Necessary as the group is created by the dependencies

* Replace Whatsapp typos with Signal

* Fix service description inconsistency in install/upgrade/restore

* comment out bot_synapse_adm for now

* remove hack for signald permissions

* deactivate encryption by default

* no encryption in manifest

* fix db_pwd upgrade

* Update config.yaml

Co-authored-by: Nathanaël HANNEBERT <[email protected]>
Co-authored-by: ericgaspar <[email protected]>
Co-authored-by: Mayeul Cantan <[email protected]>
@Gredin67 Gredin67 deleted the linter-rebased branch January 25, 2022 08:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants