Bug 9827

Summary: Remove 'insecure' system preference
Product: Koha Reporter: Bernardo Gonzalez Kriegel <bgkriegel>
Component: AuthenticationAssignee: Bernardo Gonzalez Kriegel <bgkriegel>
Status: CLOSED FIXED QA Contact: Jonathan Druart <jonathan.druart>
Severity: normal    
Priority: P5 - low CC: dpavlin, jonathan.druart, tomascohen
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 9827: remove 'insecure' system preference
Bug 9827: remove 'insecure' system preference
Bug 9827: remove 'insecure' system preference

Description Bernardo Gonzalez Kriegel 2013-03-16 06:21:19 UTC
'insecure' system preference must be removed.

Functionality enabled by this system preference is broken.
In case that is set to Allow, staff login is not possible
any more.

Related to Bug 9562
Comment 1 Bernardo Gonzalez Kriegel 2013-03-16 06:46:04 UTC Comment hidden (obsolete)
Comment 2 Tomás Cohen Arazi 2013-03-16 15:49:12 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2013-03-18 09:36:50 UTC
QA comment:

This patch removes the insecure syspref.
The 'insecure' behavior is not well implemented and some scripts raise an error if enabled.

Marked as Passed QA.
Comment 4 Jonathan Druart 2013-03-18 09:37:06 UTC
Created attachment 16211 [details] [review]
Bug 9827: remove 'insecure' system preference

This patch removes 'insecure' system preference.

Also removes remaining code that make use of
the preference. It's broken anyway.

Only remains a reference in POD of C4/Boolean.pm

To test:
1) If you like, enable 'insecure' syspref. Broken system.
WARN: be prepared to revert value in database.

2) Apply the patch

3) Run updatedatabase.pl

4) Check that Staff login proceeds as usual.

5) Check that 'insecure' syspref is no more.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Amended patch: Remove 2 occurrences of insecure (in comment only)
Comment 5 Jared Camins-Esakov 2013-03-20 01:50:01 UTC
This patch has been pushed to master.