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

Add SnappyMail webmail #1796

Closed
wants to merge 5 commits into from
Closed

Add SnappyMail webmail #1796

wants to merge 5 commits into from

Conversation

the-djmaze
Copy link
Contributor

https://snappymail.eu/

What type of PR?

Feature

What does this PR do?

Add SnappyMail as webmail option

Prerequistes

Before we can consider review and merge, please make sure the following list is done and checked.
If an entry in not applicable, you can check it or remove it from the list.

  • In case of feature or enhancement: documentation updated accordingly
  • Unless it's docs or a minor change: add changelog entry file.

@mergify
Copy link
Contributor

mergify bot commented Mar 26, 2021

Thanks for submitting this pull request.
Bors-ng will now build test images. When it succeeds, we will continue to review and test your PR.

bors try

Note: if this build fails, read this.

bors bot added a commit that referenced this pull request Mar 26, 2021
@bors
Copy link
Contributor

bors bot commented Mar 26, 2021

try

Build failed:

@parisni
Copy link
Contributor

parisni commented Jun 4, 2021

SnappyMail can be backed by postgres AFAIK ? That would be nice to optionally provide postgres then for it (as admin and roundcube do)
otherwise I failed to make it working, 500 error and the container logs :

webmail_1    | [Fri Jun 04 19:37:41.627648 2021] [core:alert] [pid 23] [client 127.0.0.1:59524] /var/www/html/.htaccess: Invalid command 'RewriteEngine', perhaps misspelled or defined by a module not included in the server configuration

restart: always
env_file: mailu.env
volumes:
- "/mailu/webmail:/data"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe mount something like /mailu/webmail/snappymail to avoid collision with other webmails ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe mount something like /mailu/webmail/snappymail to avoid collision with other webmails ?

Would be nice if i knew how.
Seems that the others (RoundCube and RainLoop) don't have that either.

the-djmaze pushed a commit to the-djmaze/snappymail that referenced this pull request Jul 14, 2021
@the-djmaze
Copy link
Contributor Author

the-djmaze commented Jul 14, 2021

SnappyMail can be backed by postgres AFAIK ? That would be nice to optionally provide postgres then for it (as admin and roundcube do)

PDO is only used when activating the Addressbook feature.
Since it is PDO you can use: mysql, pgsql or sqlite

otherwise I failed to make it working, 500 error and the container logs :

webmail_1    | [Fri Jun 04 19:37:41.627648 2021] [core:alert] [pid 23] [client 127.0.0.1:59524] /var/www/html/.htaccess: Invalid command 'RewriteEngine', perhaps misspelled or defined by a module not included in the server configuration

Thanks, i solved that issue

@Diman0
Copy link
Member

Diman0 commented Nov 12, 2021

After internal discussion we have decided not to accept this PR. Two webmails are sufficient for the project. The Mailu project always had a very low amount of active contributors. We cannot use another webmail that adds more cost of ownership. We would have to maintain this and update it for every future release. Any webmail is basically an IMAP client. You can simply run your own image of SnappyMail and connect via IMAP to Mailu.

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.

None yet

3 participants