? ? Pending

User tests: Successful: Unsuccessful:

avatar zero-24
zero-24
18 Jan 2021

Pull Request for an Issue reported by @PhilETaylor to the JSST. The JSST decided to patch this in the public tracker hence here the PR.

Summary of Changes

Make sure the sendmail endpoint can only be called via post

Testing Instructions

  • Make sure the sendmail test button works
  • apply this patch
  • Make sure the sendmail test button still works

Actual result BEFORE applying this Pull Request

The sendmail test button works

Expected result AFTER applying this Pull Request

The sendmail test button still works

Documentation Changes Required

none

avatar zero-24 zero-24 - open - 18 Jan 2021
avatar zero-24 zero-24 - change - 18 Jan 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 18 Jan 2021
Category Administration com_config
avatar toivo toivo - test_item - 19 Jan 2021 - Tested successfully
avatar toivo
toivo - comment - 19 Jan 2021

I have tested this item successfully on a20f9c0

Tested successfully in 3.9.25-dev of 19 January using PHP 8.0.1.


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

avatar gostn gostn - test_item - 20 Jan 2021 - Tested successfully
avatar gostn
gostn - comment - 20 Jan 2021

I have tested this item successfully on a20f9c0


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

avatar alikon alikon - change - 20 Jan 2021
Status Pending Ready to Commit
avatar alikon
alikon - comment - 20 Jan 2021

RTC


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

avatar richard67 richard67 - change - 20 Jan 2021
Labels Added: ? ?
avatar HLeithner HLeithner - change - 20 Jan 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-01-20 09:54:14
Closed_By HLeithner
Labels
avatar HLeithner
HLeithner - comment - 20 Jan 2021

thanks

avatar PhilETaylor
PhilETaylor - comment - 13 Apr 2021

Your comment has nothing to do with this PR.

Add a Comment

Login with GitHub to post a comment