Closed
Bug 1284263
Opened 9 years ago
Closed 9 years ago
Add optional support for $DATABASE_URL instead of localconfig for db_* connection params.
Categories
(Bugzilla :: Database, defect, P1)
Tracking
()
RESOLVED
FIXED
Bugzilla 6.0
People
(Reporter: dylan, Assigned: dylan)
References
(Blocks 2 open bugs)
Details
(Whiteboard: [heroku])
Attachments
(1 file)
I could just make the heroku buildpack parse the $DATABASE_URL at compile time, but the env can actually change. It would be better if bugzilla could directly use this information.
|
Assignee | |
Updated•9 years ago
|
Priority: -- → P1
|
||
Comment 1•9 years ago
|
||
|
|
Assignee | |
Updated•9 years ago
|
Attachment #8767704 -
Flags: review?
|
|
Assignee | |
Updated•9 years ago
|
Attachment #8767704 -
Flags: review? → review?(dkl)
|
||
Comment 2•9 years ago
|
||
Comment on attachment 8767704 [details] [review]
[bugzilla] dylanwh:bug-1284263 > bugzilla:master
:: template/en/default/setup/strings.txt.pl
+Note this requires the 'heroku' feature to be enabled.
s/heroku/db_from_env/
I was able to test this locally by setting the proper environment variables in mod_cgi and mod_perl.
r=dkl
Attachment #8767704 -
Flags: review?(dkl) → review+
|
||
Comment 3•9 years ago
|
||
First step towards http://12factor.net/config :-D
|
|
Assignee | |
Comment 4•9 years ago
|
||
To github.com:bugzilla/bugzilla.git
c61b7ce..1e2027a master -> master
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Updated•9 years ago
|
Target Milestone: --- → Bugzilla 6.0
You need to log in
before you can comment on or make changes to this bug.
Description
•