Tickets :: [#8075] Add check for the PHP HASH moduly

6.0.0-beta1

7/7/25

Summary	Add check for the PHP HASH moduly
Queue	Horde Base
Queue Version	HEAD
Type	Enhancement
State	Resolved
Priority	1. Low
Owners	jan (at) horde (dot) org
Requester	wrobel (at) horde (dot) org
Created	03/12/2009 (5961 days ago)
Due
Updated	06/16/2009 (5865 days ago)
Assigned
Resolved	03/13/2009 (5960 days ago)
Milestone
Patch	No

06/16/2009 01:28:06 PM	Jan Schneider	Comment #12	Reply to this comment
I got the problem on an OpenPKG system. The same system also lacked JSON support. So we should check for that one, too. After activating HASH and JSON Horde was running fine. JSON is a different story, because we have a fallback PHP implementation.

03/15/2009 09:13:26 AM	Gunnar Wrobel	Comment #11	Reply to this comment
I got the problem on an OpenPKG system. The same system also lacked JSON support. So we should check for that one, too. After activating HASH and JSON Horde was running fine. I also don't think we can influence the distros decisions there. In fact if you look at Gentoo it is not even a decision the package manager makes. On Gentoo the user has full control over all PHP modules by activating/deactiving the corresponding USE flags.

03/14/2009 04:53:15 PM	Jan Schneider	Comment #10	Reply to this comment
I have added --disable-hash to the error message. We already check for PCRE.

03/14/2009 04:52:46 PM	CVS Commit	Comment #9	Reply to this comment
Changes have been made in CVS for this ticket: http://cvs.horde.org/diff.php/horde/test.php?rt=horde&r1=1.178&r2=1.179&ty=u

03/13/2009 04:34:35 PM	Michael Slusarz	Comment #8	Reply to this comment
Why? The list of extensions being enabled by default in PHP is changing all the time, so why not require the extension and make this clear on the test.php page? if that's the case, then there should also be mention that PHP should not be compiled with '--disable-hash' since that will also disable the hash module. And if we are going to check these kind of things, there are other modules we have to check as well. For example, as of PHP 5.2.x it is possible to disable PCRE functions.

03/13/2009 02:10:58 PM	Jan Schneider	Comment #7	Reply to this comment
Why? The list of extensions being enabled by default in PHP is changing all the time, so why not require the extension and make this clear on the test.php page? And I don't think it's our job to tell the distro maintainers which extensions to enables. And please note that even though I have seen this happening, I just assumed it was the distro's default, I didn't digg any further.

03/13/2009 01:59:24 PM	Chuck Hagenbuch	Comment #6	Reply to this comment
Arg. Can we detect these people and send email to the distro pointing out how utterly horrible this is?

03/13/2009 10:29:20 AM	Jan Schneider	Comment #5 Assigned to Jan Schneider State ⇒ Resolved	Reply to this comment
This happened to me on a client's site too, I guess there are a few people, or even worse, several distributions, that might use --disable-all. So I tweaked the error messages and committed it.

03/13/2009 10:29:15 AM	CVS Commit	Comment #4	Reply to this comment
Changes have been made in CVS for this ticket: http://cvs.horde.org/diff.php/horde/test.php?rt=horde&r1=1.176&r2=1.177&ty=u

03/13/2009 09:58:03 AM	Gunnar Wrobel	Comment #3	Reply to this comment
The problem occured on a php-5.2.8 installation on standard Kolab server. The installation uses configure with "--disable-all" though. Guess that might have caused the problem. Thanks for the feedback!

03/12/2009 02:16:50 PM	Chuck Hagenbuch	Comment #2 State ⇒ Rejected	Reply to this comment
hash is part of PHP core after 5.1.2; Horde HEAD requires 5.2+

03/12/2009 06:12:59 AM	Gunnar Wrobel	Comment #1 Patch ⇒ No State ⇒ New New Attachment: test_php.patch Milestone ⇒ Queue ⇒ Horde Base Summary ⇒ Add check for the PHP HASH moduly Type ⇒ Enhancement Priority ⇒ 1. Low	Reply to this comment
Horde_Secret within git HEAD requires the hash() function. The availabilty of this module should be checked in the test script as the login screen won't show otherwise. Patch attached.