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

feat(maildev): log OTP in console through maildev in dev mode #1347

Merged
merged 4 commits into from
Mar 25, 2021

Conversation

orbitalsqwib
Copy link
Contributor

@orbitalsqwib orbitalsqwib commented Mar 24, 2021

Problem

Currently, the implementation for maildev does not include support for Gitpod, as maildev is run on another container instance and thus cannot be ported to Gitpod as is. Furthermore, the development workflow is slightly slowed down due to having to get the OTP manually from maildev's visual interface through localhost:1080.

Solution

By logging the OTP in the console, developers should be able to retrieve the OTP there and log into the application without having to open localhost:1080. In addition, this pull request also implements maildev for Gitpod through running a maildev instance on the same container instance.

  • logs the OTP in console for logins through maildev during development mode for convenience
  • implements maildev support for gitpod
  • refactored transport option configurations for server side config to reuse some SES variables for maildev

Deploy Notes

Environmental Variables

  • Reused SES_HOST and SES_PORT for configuring maildev host and smtp ports.
    • docker-compose.yml sets SES_HOST and SES_PORT to maildev and 25 respectively
    • gitpod.Dockerfile sets SES_HOST and SES_PORT to localhost and 1025 respectively

- logs the OTP in console for logins through maildev during development mode for convenience
- implements maildev support for gitpod
.gitpod.yml Outdated Show resolved Hide resolved
@orbitalsqwib orbitalsqwib force-pushed the feat/maildev branch 3 times, most recently from 63bff04 to 9bb6c0f Compare March 25, 2021 03:17
- Implemented code injection for the maildev executable to enable OTP logging
on new mail
- Modified the docker-run command to run a local instance of verbose maildev
- Refactored transport options to share `SES_HOST` and `SES_PORT` with maildev
Copy link
Contributor

@LoneRifle LoneRifle left a comment

Choose a reason for hiding this comment

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

lgtm

@LoneRifle LoneRifle merged commit 54be6fc into develop Mar 25, 2021
@LoneRifle LoneRifle deleted the feat/maildev branch March 25, 2021 06:40
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.

2 participants