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

mailbox is useless #7

Open
tpimh opened this issue Jul 19, 2021 · 2 comments
Open

mailbox is useless #7

tpimh opened this issue Jul 19, 2021 · 2 comments

Comments

@tpimh
Copy link

tpimh commented Jul 19, 2021

All mail is sent by the worker, but the mailbox is served by the server. Worker and server are in two different containers, so all mail generated by worker is not accessible by the server, rendering the mailbox useless (and even confusing, as I tried modifying server config for email settings and wondering why new mail is not appearing in static/email/test-emails).

@michaelbromley
Copy link
Member

Good point. Maybe it is the case that the mailbox should not even be enabled in a docker-compose project, since it is firmly intended to be a local, dev-time convenience, and I would expect it normally to be used in a context where neither the server nor worker are running in a container.

At the same time, setting "devMode: true" automatically starts the mailbox server, so that would need to be de-coupled and allows an extra option to not even serve the mailbox app if the worker is on a different server/container.

@tpimh
Copy link
Author

tpimh commented Jul 19, 2021

Maybe starting a separate web server just for the mailbox in worker container if devMode is enabled would be a better solution. Not sure if it's feasible, but maybe server and worker should have two different configs (or even different sets of dependencies: the worker is using very little of what's needed for the server, and vise versa).

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

No branches or pull requests

2 participants