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

mailto doesn't work #94

Open
cozhelp opened this issue Dec 28, 2023 · 3 comments
Open

mailto doesn't work #94

cozhelp opened this issue Dec 28, 2023 · 3 comments
Labels
needs-integration-test question Further information is requested

Comments

@cozhelp
Copy link

cozhelp commented Dec 28, 2023

I need help getting Mailrise to send emails through Gmail. I can get it to work for Apprise. I'm able to get Mailrise to work for other services like ntfy. I am using the Gmail app password like Apprise specifies.

configs:

  "*@*":
    urls:
      - mailto://myuser:[email protected]

tls:
  mode: off

@cozhelp cozhelp changed the title SMTP Doesn't work SMTP doesn't work Dec 28, 2023
@cozhelp cozhelp changed the title SMTP doesn't work mailto doesn't work Dec 28, 2023
@YoRyan
Copy link
Owner

YoRyan commented Dec 28, 2023

Try the -vv flag to turn on verbose logging. Does anything interesting show up?

@djonko
Copy link

djonko commented Jan 6, 2024

Same problem here :
I have set basic authentication without tls
My config

configs:
  ntfy:
    urls:
      - ntfy://192.168.0.7:9999/fdsjlfd
  "*@dfd.dfd.com":
    urls:
      - ntfy://192.168.0.1:8888/dfdfkdf
smtp:
  auth:
    basic:
      dfdxx: zzzz
      testpwd: xxxx
tls:
  mode: off

but I have now this error

Attaching to mailrise-server
mailrise-server | /home/mailrise/.local/lib/python3.12/site-packages/aiosmtpd/smtp.py:372: UserWarning: Requiring AUTH while not requiring TLS can lead to security vulnerabilities!
mailrise-server | warn("Requiring AUTH while not requiring TLS "
mailrise-server | [2024-01-06 14:44:53] WARNING:mail.log:auth_required == True but auth_require_tls == False
mailrise-server | /home/mailrise/.local/lib/python3.12/site-packages/aiosmtpd/smtp.py:372: UserWarning: Requiring AUTH while not requiring TLS can lead to security vulnerabilities!
mailrise-server | warn("Requiring AUTH while not requiring TLS "
mailrise-server | [2024-01-06 14:47:33] WARNING:mail.log:auth_required == True but auth_require_tls == False
mailrise-server | /home/mailrise/.local/lib/python3.12/site-packages/aiosmtpd/smtp.py:372: UserWarning: Requiring AUTH while not requiring TLS can lead to security vulnerabilities!
mailrise-server | warn("Requiring AUTH while not requiring TLS "
mailrise-server | [2024-01-06 14:49:36] WARNING:mail.log:auth_required == True but auth_require_tls == False
mailrise-server | /home/mailrise/.local/lib/python3.12/site-packages/aiosmtpd/smtp.py:372: UserWarning: Requiring AUTH while not requiring TLS can lead to security vulnerabilities!
mailrise-server | warn("Requiring AUTH while not requiring TLS "
mailrise-server | [2024-01-06 14:49:44] WARNING:mail.log:auth_required == True but auth_require_tls == False
mailrise-server | /home/mailrise/.local/lib/python3.12/site-packages/aiosmtpd/smtp.py:372: UserWarning: Requiring AUTH while not requiring TLS can lead to security vulnerabilities!
mailrise-server | warn("Requiring AUTH while not requiring TLS "
mailrise-server | [2024-01-06 14:50:25] WARNING:mail.log:auth_required == True but auth_require_tls == False
mailrise-server | /home/mailrise/.local/lib/python3.12/site-packages/aiosmtpd/smtp.py:372: UserWarning: Requiring AUTH while not requiring TLS can lead to security vulnerabilities!
mailrise-server | warn("Requiring AUTH while not requiring TLS "
mailrise-server | [2024-01-06 14:50:26] WARNING:mail.log:auth_required == True but auth_require_tls == False

it come with the latest update.
The version 1.3.0 doesn't have this issue ... I just downgraded to it 1.3.0

@YoRyan
Copy link
Owner

YoRyan commented Jan 9, 2024

Thanks for the report @djonko—I’ve reverted the commit that broke SMTP authentication. It was the fix for #98.

We already test configuration parsing, but I really should write some integration tests, so that we can avoid situations like this in the future.

@YoRyan YoRyan added question Further information is requested needs-integration-test labels Jan 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-integration-test question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants