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

Allow installation in sub-directories #8

Closed
kemenaran opened this issue Nov 16, 2016 · 16 comments · Fixed by #158 or #164
Closed

Allow installation in sub-directories #8

kemenaran opened this issue Nov 16, 2016 · 16 comments · Fixed by #158 or #164

Comments

@kemenaran
Copy link
Collaborator

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 kemenaran added the help wanted Contributions welcome label Nov 16, 2016
@kemenaran kemenaran removed the help wanted Contributions welcome label Oct 10, 2017
@kemenaran
Copy link
Collaborator Author

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 kemenaran changed the title Allow installation in sub-domains Allow installation in sub-directories Jul 16, 2018
@kemenaran
Copy link
Collaborator Author

Subpath support has been implemented in Mattermost 5.1.

@kemenaran kemenaran added enhancement help wanted Contributions welcome and removed on hold labels Jul 16, 2018
@ImaCrea
Copy link

ImaCrea commented Sep 3, 2019

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
Copy link
Collaborator Author

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

@ImaCrea
Copy link

ImaCrea commented Sep 8, 2019 via email

@ImaCrea
Copy link

ImaCrea commented Sep 12, 2019

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
Copy link
Collaborator Author

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
Copy link

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

@kemenaran
Copy link
Collaborator Author

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
Copy link
Collaborator Author

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.

@kemenaran kemenaran removed the help wanted Contributions welcome label Feb 9, 2020
@win8linux
Copy link

The install errors out with this:

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

Here is the log.

@kemenaran
Copy link
Collaborator Author

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
Copy link

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
Copy link

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
Copy link
Collaborator Author

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
Copy link
Collaborator Author

@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).

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