? ? Success

User tests: Successful: Unsuccessful:

avatar mbabker
mbabker
21 Mar 2016

Partial Pull Request for Issue #9507.

Summary of Changes

This adds a test to Joomla's install app to check for mcrypt support. On a server without it, though Joomla is installable, a degraded experience will be present (namely two-factor authentication cannot be used and several libraries (JCrypt, FOFEncryptAes, the random_compat polyfill, and the password_compat polyfill) either use different encryption libraries or fail completely).

Testing Instructions

On the overview page of the install app, in the Pre-Installation Check section, you will see a new line testing for mcrypt support. This should report accurately depending on your server's configuration.

avatar mbabker mbabker - open - 21 Mar 2016
avatar mbabker mbabker - change - 21 Mar 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 21 Mar 2016
Labels Added: ? ?
avatar brianteeman brianteeman - test_item - 21 Mar 2016 - Tested successfully
avatar brianteeman
brianteeman - comment - 21 Mar 2016

I have tested this item :white_check_mark: successfully on a236e7f


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

avatar brianteeman brianteeman - change - 21 Mar 2016
Category Installation Language & Strings
avatar brianteeman brianteeman - change - 21 Mar 2016
Labels
avatar brianteeman
brianteeman - comment - 21 Mar 2016

Would it perhaps be better to put the check in the code that enables the 2fa


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

avatar mbabker
mbabker - comment - 21 Mar 2016

There should be a check there too, yes. As I said on the other issue, this is just a very small piece of things and at least adds a generalized check that didn't exist before.

avatar brianteeman
brianteeman - comment - 21 Mar 2016

Either way - this is not a critical or a new issue and can wait until 3.5.1

On 21 March 2016 at 20:30, Michael Babker notifications@github.com wrote:

There should be a check there too, yes. As I said on the other issue, this
is just a very small piece of things and at least adds a generalized check
that didn't exist before.


You are receiving this because you commented.
Reply to this email directly or view it on GitHub
#9508 (comment)

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

avatar brianteeman
brianteeman - comment - 21 Mar 2016

Thinking about it this must go in the 2fa plugin code because a large % of
sites never use our installer and/or are upgrades from earlier releases and
the install check will be of no use to them at all

On 21 March 2016 at 20:31, Brian Teeman brian@teeman.net wrote:

Either way - this is not a critical or a new issue and can wait until 3.5.1

On 21 March 2016 at 20:30, Michael Babker notifications@github.com
wrote:

There should be a check there too, yes. As I said on the other issue,
this is just a very small piece of things and at least adds a generalized
check that didn't exist before.


You are receiving this because you commented.
Reply to this email directly or view it on GitHub
#9508 (comment)

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

avatar brianteeman brianteeman - change - 21 Mar 2016
Milestone Added:
avatar mbabker
mbabker - comment - 21 Mar 2016

To close out #9507 completely at a minimum you should have this PR merged, some kind of check in the com_users edit screens which disables the 2FA fields if the server doesn't support the requirements (and a big UI warning explaining why), and checks in the code which skip calling the functions if unsupported since you shouldn't rely on just the UI to block something. Ideally also the environmental checks that exist in the installer should be integrated into the system information view as a new tab so that information is available to the admin regardless of how they installed (plus just because you install on one server doesn't mean it will test the same on another server).

avatar brianteeman
brianteeman - comment - 21 Mar 2016

Yes we need a complete solution not just a quick fix ;)

On 21 March 2016 at 20:37, Michael Babker notifications@github.com wrote:

To close out #9507 #9507
completely at a minimum you should have this PR merged, some kind of check
in the com_users edit screens which disables the 2FA fields if the server
doesn't support the requirements (and a big UI warning explaining why), and
checks in the code which skip calling the functions if unsupported since
you shouldn't rely on just the UI to block something. Ideally also the
environmental checks that exist in the installer should be integrated into
the system information view as a new tab so that information is available
to the admin regardless of how they installed (plus just because you
install on one server doesn't mean it will test the same on another server).


You are receiving this because you commented.
Reply to this email directly or view it on GitHub
#9508 (comment)

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

avatar richard67 richard67 - test_item - 21 Mar 2016 - Tested successfully
avatar richard67
richard67 - comment - 21 Mar 2016

I have tested this item :white_check_mark: successfully on a236e7f

Successfully tested for the case of mcrypt support = true.

Pre-Installation Check section of the installation app shows this.

Unfortunately I have no other server available to test the other case where it is false.


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

avatar brianteeman brianteeman - change - 22 Mar 2016
Milestone Removed:
avatar brianteeman brianteeman - change - 22 Mar 2016
Milestone Added:
avatar brianteeman brianteeman - change - 22 Mar 2016
Milestone Added:
avatar brianteeman brianteeman - change - 22 Mar 2016
Milestone Removed:
avatar andrepereiradasilva
andrepereiradasilva - comment - 22 Mar 2016

See for adding mcrypt info to sysinfo #9532

avatar brianteeman brianteeman - change - 24 Mar 2016
Status Pending Ready to Commit
Labels
avatar brianteeman
brianteeman - comment - 24 Mar 2016

RTC


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

avatar joomla-cms-bot joomla-cms-bot - change - 24 Mar 2016
Labels Added: ?
avatar rdeutz rdeutz - reference | bd0a6eb - 12 Apr 16
avatar rdeutz rdeutz - merge - 12 Apr 2016
avatar rdeutz rdeutz - close - 12 Apr 2016
avatar rdeutz rdeutz - change - 12 Apr 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-04-12 19:45:38
Closed_By rdeutz
avatar mbabker mbabker - head_ref_deleted - 12 Apr 2016
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:
avatar brianteeman brianteeman - change - 11 May 2016
Labels Removed: ?

Add a Comment

Login with GitHub to post a comment