? ? Success

User tests: Successful: Unsuccessful:

avatar andrepereiradasilva
andrepereiradasilva
18 Oct 2016

Summary of Changes

Since the current global config "Force HTTPS" warning message doesn't say much to user, this PR makes the message show the real problem.

Testing Instructions

  • Use latest staging
  • In a server with SSL config issues try to save joomla config as Forced HTTPS
  • You will get a warning message like this

image

  • Now apply this patch and test again. you will get a more useful warning message that says why the problem exists

image

Documentation Changes Required

None.

@brianteeman please check

avatar andrepereiradasilva andrepereiradasilva - open - 18 Oct 2016
avatar andrepereiradasilva andrepereiradasilva - change - 18 Oct 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 18 Oct 2016
Labels Added: ? ?
avatar joomla-cms-bot joomla-cms-bot - change - 18 Oct 2016
Category Administration Components Language & Strings
avatar andrepereiradasilva andrepereiradasilva - change - 18 Oct 2016
The description was changed
avatar andrepereiradasilva andrepereiradasilva - edited - 18 Oct 2016
avatar joomla-cms-bot joomla-cms-bot - change - 18 Oct 2016
Labels Removed: ?
avatar andrepereiradasilva andrepereiradasilva - change - 31 Oct 2016
The description was changed
avatar conconnl
conconnl - comment - 4 Nov 2016

@andrepereiradasilva any ideas on creating config issues for HTTPS ?


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

avatar andrepereiradasilva
andrepereiradasilva - comment - 4 Nov 2016

@conconnl didn't understand your question

avatar conconnl
conconnl - comment - 4 Nov 2016

@andrepereiradasilva to be able to test this PR, you need a server with SSL configuration issues.
Any idea how to create those issues

avatar andrepereiradasilva
andrepereiradasilva - comment - 4 Nov 2016

To create an issue do this:

avatar conconnl conconnl - test_item - 8 Nov 2016 - Tested successfully
avatar conconnl
conconnl - comment - 8 Nov 2016

I have tested this item successfully on 93fd030

Tested successfully.
The test instructions in the last message are the easiest way to create a not working HTTPS situation to be able to see the error message.


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

avatar MartijnMaandag
MartijnMaandag - comment - 8 Nov 2016

I have tested this item successfully on 93fd030

Tested on Localhost:
Message:
HTTPS has not been enabled as it is not available on this server. HTTPS connection test failed with the following error: Failed to connect to localhost port 443: Connection refused


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

avatar MartijnMaandag MartijnMaandag - test_item - 8 Nov 2016 - Tested successfully
avatar jeckodevelopment jeckodevelopment - change - 8 Nov 2016
The description was changed
Status Pending Ready to Commit
Labels
avatar jeckodevelopment
jeckodevelopment - comment - 8 Nov 2016

RTC


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

avatar jeckodevelopment jeckodevelopment - change - 8 Nov 2016
Labels
avatar jeckodevelopment jeckodevelopment - edited - 8 Nov 2016
avatar zero-24 zero-24 - change - 9 Nov 2016
Milestone Added:
avatar rdeutz rdeutz - close - 10 Nov 2016
avatar rdeutz rdeutz - merge - 10 Nov 2016
avatar rdeutz rdeutz - reference | 0cfa177 - 10 Nov 16
avatar rdeutz rdeutz - merge - 10 Nov 2016
avatar rdeutz rdeutz - close - 10 Nov 2016
avatar rdeutz rdeutz - change - 10 Nov 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-11-10 09:03:37
Closed_By rdeutz
avatar andrepereiradasilva andrepereiradasilva - head_ref_deleted - 10 Nov 2016

Add a Comment

Login with GitHub to post a comment