feat: smtp port #5

Open
fauno wants to merge 2 commits from smtp-port into main
Owner

my server is starttls so i needed 587

my server is starttls so i needed 587
fauno added 1 commit 2025-04-03 20:04:32 +00:00
feat: smtp port
Some checks failed
continuous-integration/drone/pr Build is failing
84b2c3bc26
3wordchant reviewed 2025-04-03 21:42:28 +00:00
@ -4,3 +4,3 @@
export SYMFONY__ENV__SECRET=$(cat $SYMFONY__ENV__SECRET_FILE)
export SYMFONY__ENV__MAILER_DSN=smtp://${SYMFONY__ENV__MAILER_USER}:$(cat ${SYMFONY__ENV__MAILER_PASSWORD_FILE})@${SYMFONY__ENV__MAILER_HOST}:465
export SYMFONY__ENV__MAILER_DSN=smtp://${SYMFONY__ENV__MAILER_USER}:$(cat ${SYMFONY__ENV__MAILER_PASSWORD_FILE})@${SYMFONY__ENV__MAILER_HOST}:${SYMFONY__ENV__MAILER_PORT}
Owner

Could we maybe do ${SYMFONY__ENV__MAILER_PORT:-465} for backwards compatibility?

Could we maybe do `${SYMFONY__ENV__MAILER_PORT:-465}` for backwards compatibility?
fauno marked this conversation as resolved
decentral1se approved these changes 2025-04-05 05:47:16 +00:00
Dismissed
decentral1se left a comment
Owner
LGTM. https://git.coopcloud.tech/coop-cloud/wallabag/pulls/5/files#issuecomment-23259 seems like a great idea too.
fauno added 1 commit 2025-04-23 19:07:57 +00:00
fix: backwards compatibility
Some checks failed
continuous-integration/drone/pr Build is failing
290333d38b
fauno requested review from decentral1se 2025-04-23 19:08:34 +00:00
decentral1se approved these changes 2025-04-24 06:24:45 +00:00
decentral1se left a comment
Owner

LGTM

LGTM
Some checks failed
continuous-integration/drone/pr Build is failing
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin smtp-port:smtp-port
git checkout smtp-port
Sign in to join this conversation.
No description provided.