Closed (fixed)
Project:
Hosting
Version:
7.x-3.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
8 Nov 2017 at 02:46 UTC
Updated:
23 Nov 2017 at 12:49 UTC
Jump to comment: Most recent
When hosting-queued.drush.inc was ported to D7, nobody noticed that the db handling code was no longer sane under the PDO model. This causes two problems.
1) Sites that don't have mysqli module available will have a nonfunctional hosting-queued.
2) hosting-queued will leak database connections.
Since D7 has full support for database connection management, there is no need to mess with calling raw driver functions.
| Comment | File | Size | Author |
|---|---|---|---|
| hosting_queued_fix_db_handling.patch | 1.59 KB | bdragon |
Comments
Comment #2
bdragon commentedupdate attribution
Comment #3
helmo commentedLooks OK, thanks.
Comment #5
helmo commentedTests ran OK in https://gitlab.com/aegir/provision/pipelines/13746450