? Success

User tests: Successful: Unsuccessful:

avatar anibalsanchez
anibalsanchez
13 Apr 2016

Pull Request for Issue #9768 / #9803

Summary of Changes

Send Test Mail button does not correctly evaluate "SMTP Authentication" radio. Then, mails are sent via SMTP even when "SMTP Authentication" is disabled.

Fix: smtpauth : $('input[name="jform[smtpauth]"]:checked').val(),

The problem is found when your fill all SMTP credentials and keep "SMTP Authentication"=No... it applies the credentials even when it is disabled.

In this PR, I have modified the code to really check for the "SMTP Authentication" radio value.

Steps to reproduce the issue

  • Fill all SMTP fields in Mail Settings for a SMTP server that requires authentication
  • But, don't enable SMTP Authentication
  • Send Test Mail button sends emails even when the radio is disabled

Testing Instructions

  1. Fill all SMTP fields in Mail Settings for a SMTP server that requires authentication
  2. Test it, click on 'Send Test Mail' button to receive an email
  3. Disable "SMTP Authentication" radio
  4. Test it, click on 'Send Test Mail' button ... email must not be received and error is shown about the failed authentication
avatar anibalsanchez anibalsanchez - open - 13 Apr 2016
avatar anibalsanchez anibalsanchez - change - 13 Apr 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 13 Apr 2016
Labels Added: ?
avatar andrepereiradasilva
andrepereiradasilva - comment - 14 Apr 2016

Ok. I tested this.

Without patch i get. Mail is sent (i think is sent with php mail fallback - see #9680) and i get:

The email was sent successfully to config-from-email@example.com using SMTP. You should check that you've received the test email.

With patch i get. Mail is not sent and i get:

Test mail could not be sent.
SMTP Error: The following recipients failed: config-from-email@example.com: : Recipient address rejected: Access denied

So is this a success?

avatar anibalsanchez
anibalsanchez - comment - 14 Apr 2016

@andrepereiradasilva Yes, it is a success! Mail must be rejected when authentication is disabled and the server requires authentication.

avatar andrepereiradasilva andrepereiradasilva - test_item - 14 Apr 2016 - Tested successfully
avatar andrepereiradasilva
andrepereiradasilva - comment - 14 Apr 2016

I have tested this item :white_check_mark: successfully on 81e541c

as comments above


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9904.

avatar henkrijneveld henkrijneveld - test_item - 18 Apr 2016 - Tested successfully
avatar henkrijneveld
henkrijneveld - comment - 18 Apr 2016

I have tested this item :white_check_mark: successfully on 81e541c

Mail without authentication is now send with the "send test e-mail" button.

Thx for solving this, I ran into this issue earlier.


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9904.

avatar brianteeman brianteeman - change - 19 Apr 2016
Status Pending Ready to Commit
avatar brianteeman
brianteeman - comment - 19 Apr 2016

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9904.

avatar joomla-cms-bot joomla-cms-bot - change - 19 Apr 2016
Labels Added: ?
avatar brianteeman brianteeman - change - 19 Apr 2016
Milestone Added:
avatar brianteeman brianteeman - change - 19 Apr 2016
Labels
avatar brianteeman brianteeman - change - 19 Apr 2016
Category Administration
avatar rdeutz rdeutz - reference | f6655ac - 19 Apr 16
avatar rdeutz rdeutz - merge - 19 Apr 2016
avatar rdeutz rdeutz - close - 19 Apr 2016
avatar rdeutz rdeutz - change - 19 Apr 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-04-19 11:27:52
Closed_By rdeutz
Labels
avatar rdeutz rdeutz - close - 19 Apr 2016
avatar rdeutz rdeutz - merge - 19 Apr 2016
avatar joomla-cms-bot joomla-cms-bot - close - 19 Apr 2016
avatar joomla-cms-bot joomla-cms-bot - change - 19 Apr 2016
Labels Removed: ?
avatar rdeutz rdeutz - change - 1 May 2016
Milestone Removed:
avatar rdeutz rdeutz - change - 1 May 2016
Milestone Added:
avatar rdeutz rdeutz - change - 1 May 2016
Milestone Added:
avatar rdeutz rdeutz - change - 1 May 2016
Milestone Removed:

Add a Comment

Login with GitHub to post a comment