I recently used this module to test an authentication setup, only to notice that when I used wrong configurations in this module I got locked out of my site. Disabling the module helped, but without an .install file (with hook_uninstall to delete variables) I couldn't get rid of the wrong configuration variables that I had set.

CommentFileSizeAuthor
#2 webserver_auth-add-install-file-2648124-2.patch899 bytesgaards
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

DMaester created an issue. See original summary.

gaards’s picture

gaards CreditAttribution: gaards commented
FileSize
webserver_auth-add-install-file-2648124-2.patch899 bytes

Patch attached.

gaards’s picture

gaards CreditAttribution: gaards commented
Title: Module doesn't have an .install file » Module doesn't have a .install file
Manuel Garcia’s picture

Manuel Garcia CreditAttribution: Manuel Garcia as a volunteer and at Appnovation commented
Title: Module doesn't have a .install file » Module doesn't clean up its variables on uninstall
Version: 7.x-1.5 » 7.x-1.x-dev
Category: Task » Bug report
Priority: Normal » Major
Status: Needs review » Reviewed & tested by the community

Patch applies cleanly to 7.x-1.x and the hook_uninstall cleans up all variables defined by the module.

  • specky_rum committed a626bdb on 7.x-1.x authored by Maesteri 
    Issue #2648124 by Maesteri: Module doesn't clean up its variables on...
specky_rum’s picture

specky_rum CreditAttribution: specky_rum as a volunteer commented
Status: Reviewed & tested by the community » Fixed

This has been applied and is in the 1.6 release. Thanks very much and apologies that it has taken so long to commit.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.