Allow installation in sub-directories

[kemenaran]

For now the installation is only possible at the root of a domain. This is not very flexible, and prevents installing Mattermost on the main domain of a Yunohost instance.

A configuration option could be added to install Mattermost in a subdirectory if needed.

[kemenaran]

Mattermost doesn't support installation in relative directories for now.

The issue is being tracked on https://mattermost.atlassian.net/browse/PLT-200.
Upstream contributions to add support for this into Mattermost are welcome.

[kemenaran]

Subpath support has been implemented in Mattermost 5.1.

[ImaCrea]

Hello dear @kemenaran would it possible to upgrade the package with Mattermost 5.1 so we could install it on a subdirectory please?
Restriction to root domain is quiet restrictive as we can't install other apps then :/ (or maybe there's something I havent understood well).

cheers

[kemenaran]

Yes, you're right; thanks for the reminder. I'll try to work on this when I'll have spare time.

[ImaCrea]

This would be wonderful! I think it would greatly help us share discoveries and then offer even better playlists for the MailTape crew :)

…

On Sun, Sep 8, 2019 at 18:45, Pierre de La Morinerie ***@***.***> wrote: Yes, you're right; thanks for the reminder. I'll try to work on this when I'll have spare time. — You are receiving this because you commented. Reply to this email directly, [view it on GitHub](#8?email_source=notifications&email_token=AAF2TLDB6W7QCME2QEZDALLQIUT4NA5CNFSM4CWSL6P2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6FUDMI#issuecomment-529220017), or [mute the thread](https://github.com/notifications/unsubscribe-auth/AAF2TLCLYU6XCLBFHKOITLLQIUT4NANCNFSM4CWSL6PQ).

[ImaCrea]

Hi @kemenaran actually I'm playing with Synapse server and it looks like Matrix can do the same as mattermost but in a more open way. Or am I missing something?

[kemenaran]

Well, I don't know much about Synapse. For me Mattermost is nicely work-oriented, and has a bunch of integrations with other services (Github, Jira, etc) – but you're probably in the best position to see what suits you best :)

[win8linux]

Is there any progress on updating the package to 5.1 or higher @kemenaran?

[kemenaran]

The package is actually updated to Mattermost 5.19 – but I didn't took the time to implement a proper configuration for setups in subdirectories yet.

Thanks for reminding it to me though :)

[kemenaran]

Support for subpaths has been merged into the testing branch.

If you want to test installing Mattermost in a subpath, run the following command on your Yunohost box:

sudo yunohost app install https://github.com/YunoHost-Apps/mattermost_ynh/tree/testing/

Please tell me if you encounter any issues with subpath support.

[win8linux]

The install errors out with this:

Error: The operation 'Install 'mattermost' application' has failed!

Here is the log.

[kemenaran]

Hmm, from the log, that's because of the password complexity:

2020-02-09 12:42:22,637: WARNING - Error: Unable to create user. Error: User.IsValid: Your password must contain at least 10 characters made up of at least one lowercase letter, at least one uppercase letter, at least one number, and at least one symbol (e.g. "~!@#$%^&*()").,

[win8linux]

Oh. Would it be possible to prompt a user installing from terminal when they enter a password that's too simple, instead of failing outright?

[win8linux]

Alright, tried a more complex password and nothing seems to be out-of-order. I'm new to Mattermost though, so someone else who is more familiar should prolly test it more thoroughly.

[kemenaran]

Yeah, that's definitely annoying :)

Unfortunately Yunohost doesn't have a facility to re-prompt the user again. However I just checked mattermost's documentation, and there is a way to relax the password constraints. I'll open an issue for this.

In the meantime, did you manage to install the app using a sufficiently complex password?

[kemenaran]

@win8linux thanks for testing!

In the meanwhile I also configured Mattermost to relax the password complexity rules. It now requires a 8-characters password, and that's it (no special chars or whatever).