Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.When Secure Pages is enabled on a site, some modules, like Ubercart's PayPal module, need to have additional entries added to secure pages configuration else they will fail in ways that are not clear to the site administrator.
The developers of these modules are (generally) aware of the issues, and some post it in their documentation. However, it would be nice for the other modules to be smart enough to make the changes themselves in an install or update hook. Such a thing would have saved me several days (and several gray hairs) getting the PayPal WPS set up on my site.
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | securepages-hooks.patch | 867 bytes | John Franklin |
| #5 | add_securepages_path_hooks-905682-5.patch | 1.51 KB | dave_robinson |
Comments
Comment #1
dandaman CreditAttribution: dandaman commentedYeah, I'd be interested in this feature as well. I might work on some code for it; it seems what we need is a function to see if that path has been set in the Secure Pages list and then a function that makes it easy to add another page.
Comment #2
dandaman CreditAttribution: dandaman commentedOhh, and it should probably set a message to the screen just so the user is notified that a new page was added to Secure Pages.
Comment #3
John Franklin CreditAttribution: John Franklin commentedI think that is a good start.
However, does this work regardless of the order they are installed? That is, if Secure Pages is installed and then Ubercart, UC can update the Secure Pages list in their install hook. However, if Secure Pages is installed *after* Ubercart, how will UC know to register their path?
Comment #4
longwaveSubscribe. Would be very useful to have a hook that can indicate paths that should be forced HTTPS or HTTP, or that should explicitly be allowed with either protocol.
Comment #5
dave_robinson CreditAttribution: dave_robinson commentedWe had this requirement ourselves and I'm currently experimenting with the attached patch. It provides 2 hooks which allow other modules to add to the pages or ignore lists. The hooks are hook_securepages_pages and hook_securepages_ignore. The hook should return an array of additional path strings for the appropriate variable.
Aside from cleaning up the code this still needs documentation and tests.
Comment #6
longwaveFor consistency with core it might be best if this hook had compatible or similar syntax to hook_admin_paths()
http://api.drupal.org/api/drupal/modules!system!system.api.php/function/...
Perhaps TRUE should mean SSL, FALSE should mean non-SSL and NULL should mean allow either?
Comment #7
John Franklin CreditAttribution: John Franklin commentedMight I suggest a somewhat tighter patch (attached) and a plea for a patch like this one or the one in #5.
Comment #8
John Franklin CreditAttribution: John Franklin commentedMarking needs review.