YunoHost-Apps / pixelfed_ynh

The federated image shareing service Pixelfed for YunoHost
https://pixelfed.org/
GNU Affero General Public License v3.0
44 stars 15 forks source link

First ldap implementation #179

Open yalh76 opened 2 years ago

yalh76 commented 2 years ago

Problem

Solution

PR Status

Automatic tests

Automatic tests can be triggered on https://ci-apps-dev.yunohost.org/ after creating the PR, by commenting "!testme", "!gogogadgetoci" or "By the power of systemd, I invoke The Great App CI to test this Pull Request!". (N.B. : for this to work you need to be a member of the Yunohost-Apps organization)

yalh76 commented 2 years ago

!testme

yunohost-bot commented 2 years ago

Fingers crossed! Test Badge

lapineige commented 2 years ago

Thank you for working on this !

This will need quite a lot of testing, for both new and existing installations.

yalh76 commented 2 years ago

!testme

yunohost-bot commented 2 years ago

Alrighty! Test Badge

lapineige commented 2 years ago

This will need a lot of testing I guess (creating/removing user account, migrating from old Pixelfed without LDAP account, …). I have no test server for that, so if anyone is willing to do it…

yalh76 commented 2 years ago

!testme

yunohost-bot commented 2 years ago

Meow :cat2: Test Badge

lapineige commented 2 years ago

Ok, I'm not sure that including php fix is the best thing here, but let's do it your way ;) At least CI will be happy :D

yalh76 commented 2 years ago

well as we already upgraded pixelfed to php7.4, I don't think we will have more issues

yalh76 commented 2 years ago

!testme

yunohost-bot commented 2 years ago

May the CI gods be with you! Test Badge

yalh76 commented 2 years ago

!testme

yunohost-bot commented 2 years ago

:v: Test Badge

yalh76 commented 2 years ago

!testme

yunohost-bot commented 2 years ago

:carousel_horse: Test Badge

lapineige commented 1 year ago

!testme

yunohost-bot commented 1 year ago

:stuck_out_tongue_winking_eye: Test Badge

lapineige commented 1 year ago
1030018 INFO DEBUG -   - Installing spatie/db-dumper (2.21.1): Extracting archive
1030019 INFO DEBUG -   - Installing spatie/laravel-backup (6.16.5): Extracting archive
1030019 INFO DEBUG -   - Installing spatie/image-optimizer (1.6.2): Extracting archive
1030019 INFO DEBUG -   - Installing spatie/laravel-image-optimizer (1.6.4): Extracting archive
1030020 INFO DEBUG -   - Installing ezyang/htmlpurifier (v4.16.0): Extracting archive
1030020 INFO DEBUG -   - Installing stevebauman/purify (v3.0.3): Extracting archive
1030020 INFO DEBUG - Package swiftmailer/swiftmailer is abandoned, you should avoid using it. Use symfony/mailer instead.
1030021 INFO DEBUG - Generating optimized autoload files
1030021 INFO DEBUG - > Illuminate\Foundation\ComposerScripts::postAutoloadDump
1030021 INFO DEBUG - > @php artisan package:discover --ansi
1030022 INFO DEBUG -
1030022 INFO DEBUG - In User.php line 14:
1030022 INFO DEBUG -
1030023 INFO DEBUG -   Interface "LdapRecord\Laravel\Auth\LdapAuthenticatable" not found
1030024 INFO DEBUG -
1030025 INFO DEBUG -
1030025 INFO DEBUG - Script @php artisan package:discover --ansi handling the post-autoload-dump event returned with error code 1
1030025 INFO DEBUG - + ynh_die '--message=Unable to install core dependencies with Composer.'

Sounds bad :sweat_smile: